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.

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

Allow 'wagtailadmin_sprite' url to bypass 2FA #2

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

Stormheg
Copy link

@Stormheg Stormheg commented Aug 1, 2023

Wagtail attempts to load SVG sprites from this url and injects them into the browsers DOM. When this url is guarded by 2FA, the HTML of the 2FA verification page is injected into the DOM instead. This results in a duplicate form on the page.

Wagtail attempts to load SVG sprites from this url and injects them into
the browsers DOM. When this url is guarded by 2FA, the HTML of the 2FA
verification page is injected into the DOM instead. This results in a
duplicate form on the page.
@Stormheg
Copy link
Author

Stormheg commented Aug 1, 2023

Short screenrecording of the issue this is fixing

Screen.Recording.2023-08-01.at.11.59.21.AM.mov

@saevarom
Copy link

Can confirm that we have this issue as well.

@Stormheg
Copy link
Author

@saevarom if you are interested, I've created a fork with Wagtail 5.1 support for one of my clients here: https://github.com/techonomydev/wagtail-2fa/commits/v3.0.0/ (not guaranteeing this fork will remain available indefinitely, but feel free to create a fork of my fork 😉)


I'm trying to get in touch with Lab Digital, the original creators of wagtail-2fa to try and see if we can transfer maintainership to Wagtail Nest. Keep an eye on the #wagtail-nest channel on Wagtail's Slack for progress updates!

@saevarom
Copy link

OK thank you @Stormheg

Right now we feel that it is safer for our client to just package the wagtail-2fa app within their repository and then revert to a public release once the project has reached a more maintainable status.

@katdom13 katdom13 merged commit d1c0164 into torchbox-forks:master Oct 17, 2023
@katdom13
Copy link

Merged, thanks!

@Stormheg Stormheg deleted the fix/double-2fa-form branch October 17, 2023 06:04
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.

3 participants