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

[stable24] Add bruteforce protection in OauthApiController #38901

Merged
merged 1 commit into from
Jun 28, 2023
Merged

[stable24] Add bruteforce protection in OauthApiController #38901

merged 1 commit into from
Jun 28, 2023

Conversation

julien-nc
Copy link
Member

Manual backport of #38773

@julien-nc julien-nc added this to the Nextcloud 24.0.13 milestone Jun 19, 2023
@julien-nc julien-nc requested review from rullzer, AndyScherzinger, ChristophWurst, miaulalala, a team, ArtificialOwl, icewind1991 and nfebe and removed request for a team June 19, 2023 13:04
@julien-nc
Copy link
Member Author

@blizzz Should we force merge this or is the EOL flag going to be removed for stable24 at some point?

@AndyScherzinger
Copy link
Member

I think it'll be force merge 👍

@blizzz
Copy link
Member

blizzz commented Jun 28, 2023

What @AndyScherzinger says

@blizzz
Copy link
Member

blizzz commented Jun 28, 2023

integration-auth did not run on drone.

@julien-nc
add bruteforce protection in OauthApiController
42e08d6 
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc
Copy link
Member Author

@blizz It was a random failure. It has now ran and was successful.
The other failure is not related.

@julien-nc julien-nc merged commit 62c7dd0 into stable24 Jun 28, 2023
@julien-nc julien-nc deleted the backport/38773/stable24 branch June 28, 2023 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nfebe nfebe nfebe approved these changes

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@rullzer rullzer rullzer approved these changes

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

@miaulalala miaulalala Awaiting requested review from miaulalala

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl was automatically assigned from nextcloud/server-backend

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 was automatically assigned from nextcloud/server-backend

Assignees
No one assigned
Labels
enhancement feature: authentication
Projects
None yet
Milestone
Nextcloud 24.0.13
Development

Successfully merging this pull request may close these issues.
5 participants
@julien-nc @AndyScherzinger @blizzz @rullzer @nfebe