You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2020/11/09 09:00:46 UTC

[GitHub] [apisix] Firstsawyou opened a new issue #2668: bug(proxy_mirror): The request header is not original in the request after mirror

Firstsawyou opened a new issue #2668:
URL: https://github.com/apache/apisix/issues/2668


   ### Issue description
   
   When using the `proxy_mirror` plugin, I first use the proxy_mirror plugin to mirror the source request, and then add the request header information to the source request. The expected mirroring request should be consistent with the source request, but the request header information in the mirrored request is changed.
   
   ### Environment
   
   * apisix version (cmd: `apisix version`): 2.0
   * OS:
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] spacewander commented on issue #2668: bug(proxy_mirror): The request header is not original in the request after mirror

Posted by GitBox <gi...@apache.org>.
spacewander commented on issue #2668:
URL: https://github.com/apache/apisix/issues/2668#issuecomment-723880898


   Give a concrete example?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] Firstsawyou commented on issue #2668: bug(proxy_mirror): The request header is not original in the request after mirror

Posted by GitBox <gi...@apache.org>.
Firstsawyou commented on issue #2668:
URL: https://github.com/apache/apisix/issues/2668#issuecomment-723920245


   > Give a concrete example?
   
   I will add later.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] membphis commented on issue #2668: bug(proxy_mirror): The request header is not original in the request after mirror

Posted by GitBox <gi...@apache.org>.
membphis commented on issue #2668:
URL: https://github.com/apache/apisix/issues/2668#issuecomment-729526544


   it seems we can not handle this case in APISIX(it depends on Nginx core)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] membphis closed issue #2668: bug(proxy_mirror): The request header is not original in the request after mirror

Posted by GitBox <gi...@apache.org>.
membphis closed issue #2668:
URL: https://github.com/apache/apisix/issues/2668


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] spacewander commented on issue #2668: bug(proxy_mirror): The request header is not original in the request after mirror

Posted by GitBox <gi...@apache.org>.
spacewander commented on issue #2668:
URL: https://github.com/apache/apisix/issues/2668#issuecomment-724416020


   We can adjust the header like this: https://github.com/openresty/lua-nginx-module/blob/7105adaa523adedec80f0aaa13388b88d08988f8/src/ngx_http_lua_subrequest.c#L1635


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] spacewander commented on issue #2668: bug(proxy_mirror): The request header is not original in the request after mirror

Posted by GitBox <gi...@apache.org>.
spacewander commented on issue #2668:
URL: https://github.com/apache/apisix/issues/2668#issuecomment-724415179


   It is because the headers of Nginx's subrequest is just a swallow copy of its parent request:
   https://github.com/nginx/nginx/blob/a3b5ccd0566e618635303309dd8592aa4cfd7eaf/src/http/ngx_http_core_module.c#L2366
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org