Skip to content

stages/identification: refresh captcha on failure (cherry-pick #13697)#13699

Merged
BeryJu merged 1 commit intoversion-2025.2from
cherry-pick-6a89a4-version-2025.2
Mar 28, 2025
Merged

stages/identification: refresh captcha on failure (cherry-pick #13697)#13699
BeryJu merged 1 commit intoversion-2025.2from
cherry-pick-6a89a4-version-2025.2

Conversation

@gcp-cherry-pick-bot
Copy link
Contributor

Cherry-picked stages/identification: refresh captcha on failure (#13697)

This reverts commit b7beac6.

Including a Captcha stage in an Identification stage is partially to
prevent password spraying attacks. The reverted commit negated this
feature to fix a UX bug. After 6fde42a9170, the functionality can now be
reinstated.


Co-authored-by: Simonyi Gergő gergo@goauthentik.io

* refactor cleanup behavior after stage form submit

* refresh captcha on failing Identification stage

* Revert "stages/identification: check captcha after checking authentication (#13533)"

This reverts commit b7beac6.

Including a Captcha stage in an Identification stage is partially to
prevent password spraying attacks. The reverted commit negated this
feature to fix a UX bug. After 6fde42a9170, the functionality can now be
reinstated.

---------

Co-authored-by: Simonyi Gergő <gergo@goauthentik.io>
@gcp-cherry-pick-bot gcp-cherry-pick-bot bot requested review from a team as code owners March 28, 2025 13:16
@codecov
Copy link

codecov bot commented Mar 28, 2025

Codecov Report

Attention: Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.

Project coverage is 92.67%. Comparing base (92a1f7e) to head (3e78a5a).
Report is 3 commits behind head on version-2025.2.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
authentik/stages/identification/stage.py 84.61% 2 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                 @@
##           version-2025.2   #13699      +/-   ##
==================================================
- Coverage           92.72%   92.67%   -0.06%     
==================================================
  Files                 793      793              
  Lines               40334    40335       +1     
==================================================
- Hits                37399    37379      -20     
- Misses               2935     2956      +21     
Flag Coverage Δ
e2e 47.95% <15.38%> (-0.09%) ⬇️
integration 24.30% <0.00%> (-0.01%) ⬇️
unit 90.45% <84.61%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@netlify
Copy link

netlify bot commented Mar 28, 2025

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 3e78a5a
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/67e6a127948ba1000821a4de
😎 Deploy Preview https://deploy-preview-13699--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@BeryJu BeryJu merged commit cc214a0 into version-2025.2 Mar 28, 2025
77 of 85 checks passed
@BeryJu BeryJu deleted the cherry-pick-6a89a4-version-2025.2 branch March 28, 2025 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant