Code-review: location regexp matching working?

[jolly-jump]

Hi Stani,

based on #2954, where I am very sure (from the specs and the tests) that ^~ does not do any regexp matching, I came across two more lines where this is (probably) the intention:

matrix-docker-ansible-deploy/roles/custom/matrix-nginx-proxy/templates/nginx/conf.d/matrix-domain.conf.j2

Line 122 in ad23055

location ^~ /_matrix/client/(r0|v1|v3|unstable)/(logout|logout/all) {

matrix-docker-ansible-deploy/roles/custom/matrix-nginx-proxy/templates/nginx/conf.d/matrix-domain.conf.j2

Line 145 in ad23055

location ^~ /_matrix/client/(r0|v1|v3|unstable)/admin/(purge_media_cache|quarantine_media/.*) {

I could find that you introduced these lines 4 months ago for the matrix-media-repo. I cannot test this and I could not find who has this running to check if my suspicion (that these lines above do not match and thus do not work) is right.

[spantaleev]

Good catch!

Looks like various people are copy-pasting location blocks and do not really look into (or remember) what the ^~ prefix does. I've supposedly fixed it now.