We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Coming from slack-ruby/slack-ruby-client#506, it looks like Rack 2.x called rewind on RACK_INPUT for form data.
rewind
RACK_INPUT
In 2.x it used to call .rewind explicitly after reading the form data.
.rewind
https://github.com/rack/rack/blob/2-2-stable/lib/rack/request.rb#L456
And it no longer does in 3.x.
rack/lib/rack/request.rb
Line 523 in e50b72a
This causes anyone accessing http_request.body to get an empty value.
http_request.body
Was this by design? I can add a test/fix if it's not.
The text was updated successfully, but these errors were encountered:
Slack::Events::Request#verify!
Yes, this is expected, and mentioned in the upgrade guide: https://github.com/rack/rack/blob/main/UPGRADE-GUIDE.md#rackinput-is-no-longer-required-to-be-rewindable. You can use Rack::RewindableInput::Middleware if you would like the request body to be rewindable, as it was in Rack 2.
Sorry, something went wrong.
Thanks, should have checked, my bad.
No branches or pull requests
Coming from slack-ruby/slack-ruby-client#506, it looks like Rack 2.x called
rewind
onRACK_INPUT
for form data.In 2.x it used to call
.rewind
explicitly after reading the form data.https://github.com/rack/rack/blob/2-2-stable/lib/rack/request.rb#L456
And it no longer does in 3.x.
rack/lib/rack/request.rb
Line 523 in e50b72a
This causes anyone accessing
http_request.body
to get an empty value.Was this by design? I can add a test/fix if it's not.
The text was updated successfully, but these errors were encountered: