Skip to content
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.

Sign up for GitHub

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

Jump to bottom

BasicAuth session name #600

Closed
sookoll opened this issue Nov 28, 2019 · 3 comments
Closed

BasicAuth session name #600

sookoll opened this issue Nov 28, 2019 · 3 comments
Assignees
@mevdschee
Labels
enhancement

Comments

@sookoll
Copy link

sookoll commented Nov 28, 2019

I have quite unic situation. I have a site, where users can log in. Site make some ajax requests to api, which is secured by BasicAuth. And I ran into session conflict, as site send PHPSESSID and API send PHPSESSID and after that broser change it for site and SESSION got lost for user.

I know it is not ideal solution, but simplest for now. I fix it by setting session name for site. But it should be configurable in api.

"basicAuth.sessionName" option for example, defaults to PHPSESSID.
And before You do session_start(), do session_name() with that option value.
@mevdschee mevdschee self-assigned this Nov 29, 2019
@mevdschee
Copy link
Owner

Good feature request. Should not take long to implement.

mevdschee added a commit that referenced this issue Nov 29, 2019
@mevdschee
fix for #600
341fe34
@mevdschee
Copy link
Owner

mevdschee commented Nov 29, 2019 •
edited
Loading

Added to: v2.6.6 Support sessionName

Enjoy!

@sookoll sookoll mentioned this issue Nov 29, 2019
Open
@sookoll
Copy link
Author

sookoll commented Nov 29, 2019

It works! Awsome job! Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mevdschee mevdschee

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mevdschee @sookoll