Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#51772 closed defect (bug) (fixed)

Add HTTP_AUTHORIZATION RewriteRule to the default multisite .htaccess file

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

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.

6 years ago #1

Keywords has-patch added; needs-patch removed

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

#2 @TimothyBlynJacobs
6 years ago

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

#3 @pbiron
6 years ago

LGTM!

#4 @hellofromTonya
6 years ago

Version trunk → 4.4

#5 @TimothyBlynJacobs
6 years ago

Component REST API → Login and Registration
Focuses rest-api added
Version 4.4 → trunk

#6 @TimothyBlynJacobs
6 years ago

Owner set to TimothyBlynJacobs
Resolution → fixed
Status new → closed

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].

helen commented on PR #743:

6 years ago #7

Merged in https://github.com/WordPress/wordpress-develop/commit/2425a11255bc45815bdc2f8d27ec3e16d871d3e7

Note: See TracTickets for help on using tickets.

Download in other formats: