proxy_method does not support variable dereferencing

[gcnovus.myopenid.com]

See ​this email from five years ago.

The following doesn't work:

location /proxy {
  internal;
  set $proxy_to_url $upstream_http_x_proxy_to_url
  set $proxy_to_host $upstream_http_x_proxy_to_host
  set $proxy_to_method $upstream_http_x_proxy_to_method
  proxy_set_header Host $proxy_to_host;
  proxy_pass $proxy_to_url;
  proxy_method $proxy_to_method;
}

The other directives interpret the arguments as variables to dereference, but proxy_method does not. This is particularly important for the case where a request is from a browser that doesn't support PUT or DELETE, so the upstream server has to calculate the verb.

[Valentin V. Bartenev]

There are many directives that do not support variables. It's usually noted in the documentation if a directive supports variables. And I don't see such note about proxy_method.

[Maxim Dounin]

I don't think that proxy_method should support variables. It's more or less trivial to write configuration with distinct methods used in distinct locations, and variables will only complicate things.

[Casey Bisson]

Actually, I can imagine a number of reasons to change this that are unrelated to location. One example is allowing only GET requests for anonymous users, but supporting POST requests for logged in users (users who have a cookie suggesting a valid login status).

[dilaz03@…]

This problem is actual for me after switch to nginx with canonical converting HTTP-methods on X-Accel-Redirect. My usecase:

// Main entry, proxy request to connected servers over VPN. Upstream may make http
// request to connected server or return X-Accel-Redirect with server IP.
location /teleport {
    proxy_pass upstream_api;
}

location ~* ^/internal-redirect-get/(.+)/(.+) {
    internal;
    set $ip $1;
    set $uri $2;
    proxy_pass http://$ip/$uri$is_args$args;
    proxy_method GET;
}

// Duplicate config for each supported HTTP verb
location ~* ^/internal-redirect-post/(.+)/(.+) {
    // ...
    proxy_method POST;
}
location ~* ^/internal-redirect-put/(.+)/(.+) {
    // ...
    proxy_method PUT;
}
location ~* ^/internal-redirect-delete/(.+)/(.+) {
    // ...
    proxy_method DELETE;
}

I found simple solution for this problem only with lua module:

location ~* ^/internal-redirect/(.+)/(.+) {
    // Extract method from header after X-Accel-Redirect and set it for request
    rewrite_by_lua_block {
        ngx.req.set_method(ngx['HTTP_' .. ngx.var['upstream_http_x_http_method']])
    }

    // ...
}

I think this is to complicated for simple task (even with lua).

Some questions on stackoverflow:

• ​http://stackoverflow.com/questions/29693645/proxying-patch-request-to-post-with-nginx
• ​http://stackoverflow.com/questions/39320243/nginx-assing-the-value-of-a-header-to-proxy-method

I will try send patch for this enhancement and tests.

[dilaz03@…]

nginx: ​http://mailman.nginx.org/pipermail/nginx-devel/2016-October/008953.html
nginx-tests: ​http://mailman.nginx.org/pipermail/nginx-devel/2016-October/008954.html

[dilaz03@…]

Updated patches:
nginx: ​http://mailman.nginx.org/pipermail/nginx-devel/2016-October/009023.html
nginx-tests: ​http://mailman.nginx.org/pipermail/nginx-devel/2016-October/009021.html

[Maxim Dounin]

Path by Dmitry committed, ca27074f8f0f.