8000 Added method User::refresh() to clear identity cache. (#46) by PavelWeirich · Pull Request #50 · nette/security · GitHub
[go: up one dir, main page]
Skip to content

Added method User::refresh() to clear identity cache. (#46)#50

Merged
dg merged 2 commits intonette:masterfrom
MakeArt:master
Dec 31, 2020
Merged

Added method User::refresh() to clear identity cache. (#46)#50
dg merged 2 commits intonette:masterfrom
MakeArt:master

Conversation

@PavelWeirich
Copy link
Contributor
@PavelWeirich PavelWeirich commented Jul 2, 2020 •
edited
Loading

Example in BasePresenter.php:

public function startup()
{
	$this->getUser()->getStorage()->setNamespace('namespace_1');
	.
	.
	.
	$this->getUser()->refresh()->getStorage()->setNamespace('namespace_2');
}

@PavelWeirich
Copy link
Contributor Author
PavelWeirich commented Jul 17, 2020

After use, I found a problem with refresh(): each time the page loads, the session_id changes. This is caused by calling $this->storage->getIdentity() in method getIdentity() in Nette\Security\User.php.

@PavelWeirich
Update User.php
b29c394 
Fix - clear $this->authenticated
@dg dg force-pushed the master branch 2 times, most recently from df0693e to 37ca1ea Compare August 2, 2020 16:56
@dg dg force-pushed the master branch 5 times, most recently from 7543e35 to 05709ff Compare October 6, 2020 17:23
@dg dg force-pushed the master branch 3 times, most recently from d183182 to 870e30c Compare October 15, 2020 22:56
@dg dg force-pushed the master branch 4 times, most recently from 3c5a02b to 72ca785 Compare November 2, 2020 16:54
@dg dg force-pushed the master branch 3 times, most recently from f0751cd to 981bdda Compare November 25, 2020 17:50
@dg dg merged commit e74784c into nette:master Dec 31, 2020
dg pushed a commit that referenced this pull request Dec 31, 2020
@PavelWeirich @dg
User: added method refresh() to clear identity cache. (#46) (#50)
3773446
dg pushed a commit that referenced this pull request Dec 31, 2020
@PavelWeirich @dg
User: added method refreshStorage() to clear identity cache. (#46) (#50)
ffdc41b
dg pushed a commit that referenced this pull request Jan 3, 2021
@PavelWeirich @dg
User: added method refreshStorage() to clear identity cache. (#46) (#50)
1eadf80
dg pushed a commit that referenced this pull request Jan 5, 2021
@PavelWeirich @dg
User: added method refreshStorage() to clear identity cache. (#46) (#50)
3baad4b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@PavelWeirich @dg
0