WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#51772 closed defect (bug) (fixed)

Add HTTP_AUTHORIZATION RewriteRule to the default multisite .htaccess file

Reported by: pbiron's profile pbiron Owned by: timothyblynjacobs's profile 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.
4 years ago #1

  • Keywords has-patch added; needs-patch removed

Trac ticket: https://core.trac.wordpress.org/ticket/51772

#2 @TimothyBlynJacobs
4 years ago

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

#3 @pbiron
4 years ago

LGTM!

#4 @hellofromTonya
4 years ago

  • Version changed from trunk to 4.4

#5 @TimothyBlynJacobs
4 years ago

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

#6 @TimothyBlynJacobs
4 years 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.