[PM-15614] Allow Users to opt out of new device verification

[ike-kottlowski]

🎟️ Tracking

PM-15614

📔 Objective

Some users would like to be able to turn off the new device verification feature. This PR is to enable that decision.

However, turning this off will reduce overall security of your account if you do not already have 2FA enabled.

📸 Screenshots

User is able to login when dbo.User.VerifyDevices is set to false (0)

PRRh7o4c8a.mp4

Transcript

User tries to login and is met with New Device Verification prompt. dbo.User.VerifyDevices is changed to false (0); user attempts to login again and are allowed to authenticate this time. dbo.User.VerifyDevices is set back to true (1); User tries to authenticate again and is successful again because the device has been saved as a knowndevice.

⏰ 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 – 5e4784e3-ced6-4e85-9673-3ce396967f2e

New Issues

Severity	Issue	Source File / Package	Checkmarx Insight
	CSRF	/src/Api/Auth/Controllers/AccountsController.cs: 967	Attack Vector
	CSRF	/src/Api/Auth/Controllers/AccountsController.cs: 975	Attack Vector
	Log_Forging	/src/Api/Auth/Controllers/AccountsController.cs: 967	Attack Vector

Fixed Issues

Severity	Issue	Source File / Package
	CSRF	/src/Api/Auth/Controllers/AccountsController.cs: 967
	CSRF	/src/Api/NotificationCenter/Controllers/NotificationsController.cs: 67
	CSRF	/src/Api/NotificationCenter/Controllers/NotificationsController.cs: 61
	Log_Forging	/src/Api/Auth/Controllers/AccountsController.cs: 967

[codecov]

Codecov Report

Attention: Patch coverage is 73.68421% with 5 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@eb9a061). Learn more about missing BASE report.
Report is 14 commits behind head on main.

Files with missing lines	Patch %	Lines
...dentityServer/RequestValidators/DeviceValidator.cs	37.50%	4 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5176   +/-   ##
=======================================
  Coverage        ?   43.34%           
=======================================
  Files           ?     1459           
  Lines           ?    66510           
  Branches        ?     6082           
=======================================
  Hits            ?    28831           
  Misses          ?    36387           
  Partials        ?     1292           

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