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

[BUG]: Can't read session vars provided by another app #14688

Closed
oleghind opened this issue Jan 6, 2020 · 1 comment
Closed

[BUG]: Can't read session vars provided by another app #14688

oleghind opened this issue Jan 6, 2020 · 1 comment
Assignees
@niden
Labels
bug A bug report status: medium Medium

Comments

@oleghind
Copy link

oleghind commented Jan 6, 2020

I am trying to read session vars provided by another app, but phalcon always adds "#" prefix while reading values from $_SESSION although I did not set any unique id.

I think "getUniqueKey" function of Manager.zep should only add "#" if uniqueId is not empty
@niden niden added 4.0.1 bug A bug report status: medium Medium labels Jan 6, 2020
@niden niden mentioned this issue Jan 6, 2020
Merged
5 tasks
@niden niden self-assigned this Jan 6, 2020
niden added a commit that referenced this issue Jan 6, 2020
@niden
[#14688] - If the uniqueId is not set return the session key as is
c01961c
niden added a commit that referenced this issue Jan 6, 2020
@niden
[#14688] - Added tests
7fa1c24
niden added a commit that referenced this issue Jan 6, 2020
@niden
[#14688] - Updated changelog
88af141
@niden
Copy link
Member

niden commented Jan 6, 2020

Resolved in #14689

@oleghind thanks for reporting this

@niden niden closed this as completed Jan 6, 2020
niden added a commit that referenced this issue Jan 6, 2020
@niden
[#14688] - If the uniqueId is not set return the session key as is
0efda8d
niden added a commit that referenced this issue Jan 6, 2020
@niden
[#14688] - Added tests
3d40e8e
niden added a commit that referenced this issue Jan 6, 2020
@niden
[#14688] - Updated changelog
3ae0c66
@niden niden moved this to Released in Phalcon v5 Aug 25, 2022
@niden niden added this to Phalcon v5 Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niden niden

Labels
bug A bug report status: medium Medium
Projects
Phalcon v5
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@niden @oleghind