[PM-13212] Fix broken styles/scripts due to FIDO2 content script

[Jamie0]

🎟️ Tracking

#11428

📔 Objective

Add document.currentScript.parentNode.removeChild(document.currentScript) to FIDO2 content script, and make the <script /> tag explicitly synchronous. This should ensure the FIDO2 <script /> tag is removed before any scripts run on the page itself, ensuring it is never the first child of documentElement.

⏰ Reminders before review

• Contributor guidelines followed
• All formatters and local linters executed and passed
• Written new unit and / or integration tests where applicable
• Protected functional changes with optionality (feature flags)
• Used internationalization (i18n) for all UI strings
• CI builds passed
• Communicated to DevOps any deployment requirements
• Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

• 👍 (:+1:) or similar for great changes
• 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
• ❓ (:question:) for questions
• 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
• 🎨 (:art:) for suggestions / improvements
• ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
• 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
• ⛏ (:pick:) for minor or nitpick changes

[bitwarden-bot]

Thank you for your contribution! We've added this to our internal Community PR board for review.
ID: PM-13212

[cagonzalezcs]

Hmm, this approach to cleaning up the script is new to me. This seems to work effectively in conjunction with other websites that were having issues with the script cleanup process we originally had in place.

I'm approving this work and moving it to QA for testing...

@Jamie0 Thank you very much for the suggestion!

[github-actions]

Checkmarx One – Scan Summary & Details – 2066f8ca-1e4c-4723-bb16-09b8f6203ca6

No New Or Fixed Issues Found

[cagonzalezcs]

Ran into a couple of issues with the implementation, please let me know if you have any questions.

[cagonzalezcs]

Once again, thank you @Jamie0 for the suggestion.

We've incorporated this work within #11444. Closing this PR as a result.