Skip to content

client/ws: fix client origin headers for ws requests#391

Merged
l5yth merged 5 commits intomainfrom
l5y-client-ws
Nov 3, 2025
Merged

client/ws: fix client origin headers for ws requests#391
l5yth merged 5 commits intomainfrom
l5y-client-ws

Conversation

@l5yth
Copy link
Collaborator

@l5yth l5yth commented Nov 3, 2025

  • client/ws: fix origin headers
  • spec: add ws client spec tests for origin headers

@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

❌ Patch coverage is 97.38806% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 99.22%. Comparing base (b6e0100) to head (8ff9fe8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lib/eth/client/ws.rb 85.71% 4 Missing ⚠️
spec/eth/client/ws_spec.rb 70.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #391      +/-   ##
==========================================
+ Coverage   99.19%   99.22%   +0.02%     
==========================================
  Files          91       91              
  Lines        6732     6823      +91     
==========================================
+ Hits         6678     6770      +92     
+ Misses         54       53       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@l5yth l5yth added enhancement New feature or request spec-tests labels Nov 3, 2025
@l5yth l5yth merged commit 4202fe3 into main Nov 3, 2025
7 of 8 checks passed
@l5yth l5yth deleted the l5y-client-ws branch November 3, 2025 10:17
ShiningRay added a commit to ShiningRay/eth.rb that referenced this pull request Dec 24, 2025
* commit 'ab969176faf033eb0a6387b875ee3aa677c4bacf':
  build(deps): bump JamesIves/github-pages-deploy-action (q9f#395)
  build(deps): bump actions/checkout from 5 to 6 (q9f#394)
  add JSONRPC error code to Client::RpcError (q9f#392)
  Update README.md (q9f#393)
  build(deps): bump github/codeql-action from 3 to 4 (q9f#390)
  client/ws: fix client origin headers for ws requests (q9f#391)
  Rename websocket client to Ws and expand websocket tests (q9f#389)
  Add WebSocket transport to client (q9f#388)
  Support tuple arrays in ABI coder (q9f#387)
  Format EIP-4844 transaction constant (q9f#386)
  Fix ABI tuple handling for ethers test cases (q9f#385)
  Increase blob throughput limits (q9f#363)
  feat: use httpx for http requests (q9f#377) (q9f#382)
  fix: transform hash keys to strings in encoder for component type compatibility (q9f#378)
  fix: improve error handling for missing keys in component encoding (q9f#376)
  build(deps): bump actions/checkout from 4 to 5 (q9f#379)

# Conflicts:
#	eth.gemspec
#	lib/eth/client/http.rb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request spec-tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant