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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor/auth security #3590
Refactor/auth security #3590
Conversation
export const CURRENT_USER = BindingKey.create<UserProfile | undefined>( | ||
'authentication.currentUser', | ||
); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep AuthenticationBindings.CURRENT_USER
as an alias to SecurityBinding.CURRENT_USER
:
export const CURRENT_USER = SecurityBindings.CURRENT_USER;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jannyHou ^^^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@raymondfeng sure, added it.
@@ -13,6 +12,7 @@ import { | |||
authorize, | |||
Authorizer, | |||
} from '../..'; | |||
import {SecurityBindings, securityId} from '../../../../security'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This need to be fixed to use @loopback/security
.
b20c568
to
07a9c58
Compare
01972ad
to
30a4965
Compare
BREAKING CHANGE: PR#3590 rewrote type UserProfile, add the missing breaking change tag here to amend the one in #3590
A follow-up PR to update
@loopback/authentication
and@loopback/authorization
to depend on@loopback/security
Tasks:
@loopback/authentication
depends on@loopback/security
@loopback/authorization
depends on@loopback/security
Checklist
馃憠 Read and sign the CLA (Contributor License Agreement) 馃憟
npm test
passes on your machinepackages/cli
were updatedexamples/*
were updated馃憠 Check out how to submit a PR 馃憟