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.

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

Apparently connected to someone else session #2168

Closed
BadrAly opened this issue Jul 14, 2021 · 1 comment · Fixed by #2285
Closed

Apparently connected to someone else session #2168

BadrAly opened this issue Jul 14, 2021 · 1 comment · Fixed by #2285
Labels
bug Breaks something but is not blocking f: user management p-High High priority (to be solved in the 2-3 next months) production Issue specifically spotted on the production server

Comments

@BadrAly
Copy link

BadrAly commented Jul 14, 2021

I was logged in as the RERO+ system librarian this morning and I clicked the refresh button of the navigator. I found myself logged in as [patron.pid:151715], which is not me.

This wrong session does not allow to access to someone else's patron account.

I was not able to duplicate it the problem, but we received comments from other persons having experienced it.

@BadrAly BadrAly added triage bug Breaks something but is not blocking labels Jul 14, 2021
@iGormilhit iGormilhit added production Issue specifically spotted on the production server f: user management and removed triage labels Jul 14, 2021
@pronguen pronguen added the p-Medium Medium priority (to be solved within 1-2 years) label Jul 14, 2021
@iGormilhit iGormilhit changed the title Connect to someone else session Apparently connected to someone else session Jul 15, 2021
@pronguen pronguen added p-High High priority (to be solved in the 2-3 next months) and removed p-Medium Medium priority (to be solved within 1-2 years) labels Jul 15, 2021
@iGormilhit
Copy link

May be related to cookies issues. A librarian said that when removing the cookies in the browser, the issue is resolved. It reminds him an issue that existed with Explore.

jma added a commit to jma/rero-ils that referenced this issue Aug 11, 2021
* Builds the menu during the first request.
* Store the logged user name in the session.
* Makes the translations and the logged user dynamic.
* Puts the menu generation code in a specific file.
* Closes: rero#2168.

Co-Authored-by: Johnny Mariéthoz <[email protected]>
@jma jma mentioned this issue Aug 11, 2021
7 tasks
@jma jma closed this as completed in #2285 Aug 11, 2021
jma added a commit that referenced this issue Aug 11, 2021
* Builds the menu during the first request.
* Store the logged user name in the session.
* Makes the translations and the logged user dynamic.
* Puts the menu generation code in a specific file.
* Closes: #2168.

Co-Authored-by: Johnny Mariéthoz <[email protected]>
jma added a commit that referenced this issue Aug 11, 2021
* Builds the menu during the first request.
* Store the logged user name in the session.
* Makes the translations and the logged user dynamic.
* Puts the menu generation code in a specific file.
* Closes: #2168.

Co-Authored-by: Johnny Mariéthoz <[email protected]>
jma added a commit that referenced this issue Aug 11, 2021
* Builds the menu during the first request.
* Store the logged user name in the session.
* Makes the translations and the logged user dynamic.
* Puts the menu generation code in a specific file.
* Closes: #2168.

Co-Authored-by: Johnny Mariéthoz <[email protected]>
jma added a commit that referenced this issue Aug 11, 2021
* Builds the menu during the first request.
* Store the logged user name in the session.
* Makes the translations and the logged user dynamic.
* Puts the menu generation code in a specific file.
* Closes: #2168.

Co-Authored-by: Johnny Mariéthoz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Breaks something but is not blocking f: user management p-High High priority (to be solved in the 2-3 next months) production Issue specifically spotted on the production server
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants