Skip to content

Limit the length of app password names - #31658

Merged
nickvergessen merged 2 commits into
masterfrom
bugfix/noid/limit-token-names
Mar 23, 2022
Merged

Limit the length of app password names#31658
nickvergessen merged 2 commits into
masterfrom
bugfix/noid/limit-token-names

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Signed-off-by: Joas Schilling coding@schilljs.com

@nickvergessennickvergessen added this to the Nextcloud 24 milestone Mar 22, 2022
@nickvergessen
nickvergessen requested review from a team, PVince81, blizzz, icewind1991, julien-nc and miaulalala and removed request for a teamMarch 22, 2022 09:55
Comment threadapps/settings/lib/Controller/AuthSettingsController.php
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the bugfix/noid/limit-token-names branch from a74cd17 to 343476fCompareMarch 23, 2022 10:01
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

adjusted the tests

@nickvergessen
nickvergessen merged commit c1215f5 into masterMar 23, 2022
@nickvergessen
nickvergessen deleted the bugfix/noid/limit-token-names branch March 23, 2022 13:12
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable21

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@nickvergessen@icewind1991@blizzz@eastyu@miaulalala