-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
aws-sdk-core-3.110.0/lib/seahorse/client/h2/connection.rb throws /http/2/compressor.rb:333:in `initialize': wrong number of arguments (given 1, expected 0) #2455
Comments
Seems this is possibly an issue with Ruby3 compatibility? - filed a bug here: ostinelli/net-http2#44 |
Please note that aws-sdk-ruby doesn't use https://github.com/ostinelli/net-http2, rather https://github.com/igrigorik/http-2. |
Ah, my bad, filed the bug in the correct place: igrigorik/http-2#158 |
In case it helps, I've forked http-2 more than one year ago, and have kept it API-compatible and up-to-date (it supports ruby 3 for 3/4 weeks). The reasons for having forked it are outlined here. It's also a transitive dependency of httpx, which also supports streaming. |
Ah, interesting, we recently switched to Faraday which seems to be well supported, is there an argument to switch from http-2 to faraday (or similar)? |
It's apples and oranges in this case, Back to the original thread, which is the require "http/2/next"
HTTP2 = HTTP2Next and see if that fixes your problem. |
A new version of http-2 has been published that resolves this issue. |
|
Confirm by changing [ ] to [x] below to ensure that it's a bug:
Describe the bug
This is my code:
It throws the following exception:
Gem name ('aws-sdk', 'aws-sdk-resources' or service gems like 'aws-sdk-s3') and its version
aws-sdk version 3.2.3
Version of Ruby, OS environment
To Reproduce (observed behavior)
Provided steps above to reproduce - it seems like net-http2 made some API changes?
Expected behavior
It shouldn't throw an exception
The text was updated successfully, but these errors were encountered: