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
Sets omniauth.headers on the request phase #806
Conversation
Hey, new to Travis, but it seems all tests are ok but there are some code offenses not introduced by myself. |
@waiting-for-dev How's your PR going? Your PR is pretty helpful for many people. |
Hey, THIS is the pull request and it has been ready for a long time waiting for some feedback ;) |
a5c7282
to
76feeea
Compare
it 'sets omniauth.headers on the request phase' do | ||
OmniAuth.config.mock_auth[:test] = {} | ||
|
||
strategy.call(make_env('/auth/test', 'HTTP_FOO' => 'bar')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mind testing that it doesn't pick up env's that aren't prepended with 'HTTP_'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, this PR was just intended to keep header values, which in rack env are always prepended with HTTP_
.
@waiting-for-dev Would you mind rebasing on Master? |
Hey @tmilewski . Rebase done. Please, review it is correct, it has been a long time since I visited it. |
Add
env['omniauth.headers']
which is a hash with header values on the request phase. Closes #804