WordPress.org

Make WordPress Core

Opened 2 weeks ago

Closed 12 days ago

Last modified 10 days ago

#51772 closed defect (bug) (fixed)

Add HTTP_AUTHORIZATION RewriteRule to the default multisite .htaccess file

Reported by: pbiron Owned by: TimothyBlynJacobs
Milestone: 5.6 Priority: normal
Severity: normal Version: 5.6
Component: Login and Registration Keywords: has-patch
Focuses: multisite, rest-api Cc:

Description

This is a follow to #42790.

Among other changes (in [49109]) a RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}] line was added to the default single site .htaccess file.

The same rewrite rule should be added to the default multisite .htaccess file (generated in network_step2()).

Change History (7)

This ticket was mentioned in PR #743 on WordPress/wordpress-develop by TimothyBJacobs.


2 weeks ago

  • Keywords has-patch added; needs-patch removed

#2 @TimothyBlynJacobs
2 weeks ago

Thanks for catching that @pbiron! Could you verify that appears in the network setup step?

#3 @pbiron
2 weeks ago

LGTM!

#4 @hellofromTonya
12 days ago

  • Version changed from trunk to 4.4

#5 @TimothyBlynJacobs
12 days ago

  • Component changed from REST API to Login and Registration
  • Focuses rest-api added
  • Version changed from 4.4 to trunk

#6 @TimothyBlynJacobs
12 days ago

  • Owner set to TimothyBlynJacobs
  • Resolution set to fixed
  • Status changed from new to closed

In 49607:

App Passwords: Include authorization rewrite rule in default multisite .htaccess

This rule was previously added to WP_Rewrite, but multisite uses a different .htaccess generation mechanism.

Props pbiron.
Fixes #51772.
See [49109].

Note: See TracTickets for help on using tickets.