Implement speculative loading considerations for safer behavior#1784
Implement speculative loading considerations for safer behavior#1784felixarntz merged 8 commits intotrunkfrom
Conversation
…etty permalinks are enabled.
…Ls like wp-activate.php, wp-signup.php, etc.
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
westonruter
left a comment
There was a problem hiding this comment.
The readme should probably be updated to note the new restrictions, in particular for the authentication requirement.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## trunk #1784 +/- ##
==========================================
+ Coverage 57.42% 57.48% +0.06%
==========================================
Files 84 84
Lines 6506 6516 +10
==========================================
+ Hits 3736 3746 +10
Misses 2770 2770
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Summary
Fixes #1157
Relevant technical choices
wp-adminandwp-login.php./wp-login.phpURL exclusion to/wp-*.php, as there are other files in the WordPress root directory that may be directly accessed and are safer to exclude, e.g.wp-activate.phpandwp-signup.php(relevant mostly for Multisite).