[CL-506] Upgrade to Angular 18

[vleague2]

🎟️ Tracking

CL-506

📔 Objective

This PR upgrades our version of Angular to 18, including all peer modules.

One of the updated modules is ng-select, which had a major version update. This required an update to the Content Security Policy for the web app, which normally blocks inline style blocks but has exceptions for specific styles. These exceptions take the form of sha256 hashes, which are basically encoded versions of the exact harcoded style block. ng-select must have had changes to their stylesheet, so we needed to update the hash.

• Reference materials: previous PR that added the hash, ng-select issue about their use of inline styles and its CSP consequences, and MDN article about using hashes for CSP
• The hash is retrieved from the Chrome console, which logs an error when you try to view a page with disallowed inline styles. It provides the hash to be added to the CSP.

⏰ 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

[github-actions]

Checkmarx One – Scan Summary & Details – 68cc7c37-0c56-446f-b16a-81b049b6c4ad

No New Or Fixed Issues Found

[codecov]

Codecov Report

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

Project coverage is 33.74%. Comparing base (c074543) to head (6173262).

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
apps/web/src/app/app.module.ts	0.00%	1 Missing ⚠️
bitwarden_license/bit-web/src/app/app.module.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #12218      +/-   ##
==========================================
+ Coverage   33.73%   33.74%   +0.01%     
==========================================
  Files        2918     2918              
  Lines       90994    90994              
  Branches    17200    17200              
==========================================
+ Hits        30693    30703      +10     
+ Misses      57901    57891      -10     
  Partials     2400     2400              

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

[willmartian]

Ty!

[vleague2]

@JimmyVo16 Sorry for the ping -- the file that your team owns is just a temporary change to make QA easier, and will be reverted before this is merged, so no need to review!

[Hinton]

This will require a note to be added to our release cards to ensure the prod rules are updated pre-release.

[vleague2]

Ty for this comment -- I am now working with sre to get the rules updated for the QA environments and for prod!