-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Reject duplicate AccessToken names #10994
Reject duplicate AccessToken names #10994
Conversation
Ready for Review |
@lafriks done |
UPDATE: need some fixes for tests |
Signed-off-by: 6543 <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #10994 +/- ##
=======================================
Coverage 43.48% 43.49%
=======================================
Files 597 597
Lines 84706 84729 +23
=======================================
+ Hits 36833 36851 +18
- Misses 43332 43337 +5
Partials 4541 4541
Continue to review full report at Codecov.
|
Make lgtm work |
@lafriks sadly there is only one review :( |
Yeah, only later noticed :) |
Having thought long about this, I think it's probably reasonable to disallow duplicate names - because it could be confusing for users. Therefor LGTM. |
make lgtm to me again ;) |
* make sure duplicate token names cannot be used * add check to api routes too * add @lunny s suggestion * fix & don't forget User.ID * AccessTokenByNameExists() return error too * unique token for each test * fix lint Signed-off-by: 6543 <[email protected]> Co-authored-by: Lanre Adelowo <[email protected]>
as title
source: #7135