Skip to content

fix: remove useless query string match and disable buffering for artifacts

Chieh-Min Wang requested to merge chiehmin18/omnibus-gitlab:nginx-location-match-git-info-refs into master

the nginx location block does not match a URL query string which results the .git/info/refs is never matched

See: https://serverfault.com/questions/811912/can-nginx-location-blocks-match-a-url-query-string

this patch makes following changes:

1. Location block does not match a URL query string which results the
   .git/info/refs being never matched. To match the config of gitlab.com,
    we simply remove the pattern here.

2. Disable buffering for artifacts.

3. Escape the backslashes to make sure "." (dot) symbols is matched
   correctly during nginx location regex matching.
Edited by Chieh-Min Wang

Merge request reports