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

Add a secureSameUser method to throw when passed a different user #29

Merged
merged 1 commit into from
Mar 10, 2021
Merged

Add a secureSameUser method to throw when passed a different user #29

merged 1 commit into from
Mar 10, 2021

Conversation

elpete
Copy link
Contributor

@elpete elpete commented Mar 9, 2021

Just as sameUser returns false when the passed in user is different than the logged in user, secureSameUser will throw when the passed in user is different than the logged in user.

@elpete elpete requested a review from lmajano March 9, 2021 01:57
@elpete elpete force-pushed the secure_same_user branch from c5a00aa to 2d2eee1 Compare March 9, 2021 01:58
@elpete
Add a secureSameUser method to throw when passed a different user
a068895 
Just as `sameUser` returns `false` when the passed in user is different than the logged in user, `secureSameUser` will throw when the passed in user is different than the logged in user.
@elpete elpete force-pushed the secure_same_user branch from 2d2eee1 to a068895 Compare March 9, 2021 01:59
@lmajano lmajano merged commit 1f689e6 into development Mar 10, 2021
@lmajano lmajano deleted the secure_same_user branch March 29, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lmajano lmajano Awaiting requested review from lmajano

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
@elpete @lmajano