-
Notifications
You must be signed in to change notification settings - Fork 18
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
Self-host select2 and remove it from CSP exceptions #2826
Comments
@abroddrick a reminder to refine this ticket. You can remove the "refinement" label afterwards. |
Cameron to refine |
@h-m-f-t I added updated this to mention that there should be a timebox of 2 days and updated my previous comment to not specifically call out a few mins of research. Also, I added in an AC that the person make a dev ticket with the proposed solution. No objections to this one as is. |
@abroddrick I've substantially revised and re-titled this issue, moving it from general concern to a specific action. Let me know what you think. |
Issue description
In #2649, we added an small external dependency. @Matt-Spence raised a concern about this in a comment, and I agree that if we want to use select2, we should in-source it and self-host.
A recent WH memo also requires us to avoid this:
I think our use of Andi on ssa.gov is justifiable, but let's remove the CSP cutout for select2.
Acceptance criteria
a few minsat least an hour trying to look for alternative solutions."Additional context
Slack thread
Links to other issues
No response
The text was updated successfully, but these errors were encountered: