Skip to content

Laravel headers #41

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

Closed
uniring opened this issue May 31, 2016 · 5 comments
Closed

Laravel headers #41

uniring opened this issue May 31, 2016 · 5 comments

Comments

@uniring
Copy link

uniring commented May 31, 2016

We've seen that request headers persist through different requests. For example, we're sending the Authentication header from one browser and getting an authenticated response from another without providing the header (we are using php-pm with only one worker to test this).

We're using Laravel 5.2 with Dingo.

Any help to fix this would be appreciated.

Thanks,

@duxet
Copy link

duxet commented Jun 1, 2016

I have fixed this issue in PR #39 - but there still might be some bugs, as Laravel contains a lot of global and static things, that doesn't play well with such architecture.

@OSDDQD
Copy link

OSDDQD commented Jun 7, 2016

@duxet doesnt work.
I'm using JWTAuth and sending Authorization header.

@andig
Copy link
Contributor

andig commented Sep 16, 2016

I've meanwhile ported my app to JWT and will try to verify this.

@andig andig assigned andig and unassigned andig Sep 16, 2016
@dzubchik
Copy link
Contributor

dzubchik commented Dec 6, 2016

This php-pm/php-pm@9b63b99 should fix an issue.

@andig
Copy link
Contributor

andig commented Jan 29, 2018

Closing due do missing feedback. Please feel free to reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants