1126315 - Add rewrite rule to htaccess to properly map BzAPI requests to the BzAPI compatibility extension path

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: API, defect)

Description

[David Lawrence [:dkl]]

This bug is to implement the idea proposed in bug 1105122 comment 19 to allow /latest/, /1.2/, or /1.3/ requests to be remapped to /bzapi/ so that the requests use the BzAPI extension.

Currently the api-dev redirect is using the following logic to do the mapping:

if(
   string.startsWith( $path, "/latest/") ||
   string.startsWith( $path, "/1.3/") ||
   string.startsWith( $path, "/1.2/")
   ) {
   $newpath = string.regexsub( $path, "^/(latest|1.2|1.3)/", "/bzapi/" );
   http.setPath( $newpath );
   http.changesite( "https://bugzilla.mozilla.org" );
} else {
      connection.discard();
}

We can do similar on BMO directly in the htaccess file by adding the following line:

RewriteRule ^(latest|1\.2|1\.3)/(.*)$ bzapi/$1 [NE] 

The api-dev can just be redirected to BMO without the need to check the path before (bug 1098342).

Patch coming up

dkl

Comment 1

[David Lawrence [:dkl]]

Attachment: 1126315_1.patch

Comment 2

[:glob ✱]

Comment on attachment 8555616 [details] [diff] [review]
1126315_1.patch

Review of attachment 8555616 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob

Comment 3

[David Lawrence [:dkl]]

Thanks!

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   66ed543..a496214  master -> master