-
Notifications
You must be signed in to change notification settings - Fork 400
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
Consider Firefox < 128 and != 115 as incompatible due to root CA expiration #13502
Conversation
…ration For now, this only applies to compatibility checks, so this will cause the add-on detail pages to show the "You need an updated version of Firefox" warning and button to show up instead of the install button for those users.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #13502 +/- ##
=======================================
Coverage 98.28% 98.28%
=======================================
Files 267 267
Lines 10610 10621 +11
Branches 3233 3237 +4
=======================================
+ Hits 10428 10439 +11
Misses 169 169
Partials 13 13 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, that looks good!
One day we'll clean up these old and inconsistent user agent strings in the tests...
Yeah I updated some of them that were in my way but there are a lot of them left... |
…ration (#13502) * Consider Firefox < 128 and != 115 as incompatible due to root CA expiration For now, this only applies to compatibility checks, so this will cause the add-on detail pages to show the "You need an updated version of Firefox" warning and button to show up instead of the install button for those users.
For now, this only applies to compatibility checks, so this will cause the add-on detail pages to show the "You need an updated version of Firefox" warning and button to show up instead of the install button for those users.
Fixes mozilla/addons#15475