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

[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

Comments

@oleghind
Copy link

@oleghind 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 this to Working on it in Phalcon Roadmap Jan 6, 2020
niden added a commit to niden/cphalcon that referenced this issue Jan 6, 2020
…s is
niden added a commit to niden/cphalcon that referenced this issue Jan 6, 2020
niden added a commit to niden/cphalcon that referenced this issue Jan 6, 2020
@niden niden mentioned this issue Jan 6, 2020
4 of 5 tasks complete
@niden niden self-assigned this Jan 6, 2020
niden added a commit to niden/cphalcon that referenced this issue Jan 6, 2020
…s is
niden added a commit to niden/cphalcon that referenced this issue Jan 6, 2020
niden added a commit to niden/cphalcon that referenced this issue Jan 6, 2020
niden added a commit that referenced this issue Jan 6, 2020
niden added a commit that referenced this issue Jan 6, 2020
@niden

This comment has been minimized.

Copy link
Member

@niden niden commented Jan 6, 2020

Resolved in #14689

@oleghind thanks for reporting this

@niden niden closed this Jan 6, 2020
Phalcon Roadmap automation moved this from Working on it to Implemented Jan 6, 2020
niden added a commit that referenced this issue Jan 6, 2020
niden added a commit that referenced this issue Jan 6, 2020
niden added a commit that referenced this issue Jan 6, 2020
@niden niden moved this from Implemented to Released in Phalcon Roadmap Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Phalcon Roadmap
  
Released
2 participants
You can’t perform that action at this time.