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 2022/03/14 01:40:32 UTC

[GitHub] [apisix] crazyMonkey1995 opened a new issue #6602: request help: real_ip_recursive for real-ip plugin

crazyMonkey1995 opened a new issue #6602:
URL: https://github.com/apache/apisix/issues/6602


   ### Issue description
   
   I found that there was once a PR about real_ip_recursive: https://github.com/apache/apisix/pull/4332
   But it seems that the real-ip plugin does not have recursive related configuration items. In fact, I looked at the real-ip code and it seems that there is no related implementation. Is there any other way to achieve it? The current scenario is to configure it in the global plugin. real-ip for dynamic modification, looking forward to reply, thanks!
   
   ### Environment
   
   - apisix version (cmd: `apisix version`): latest
   - OS (cmd: `uname -a`):
   - OpenResty / Nginx version (cmd: `nginx -V` or `openresty -V`):
   - etcd version, if have (cmd: run `curl http://127.0.0.1:9090/v1/server_info` to get the info from server-info API):
   - apisix-dashboard version, if have:
   - the plugin runner version, if the issue is about a plugin runner (cmd: depended on the kind of runner):
   - luarocks version, if the issue is about installation (cmd: `luarocks --version`):
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] crazyMonkey1995 commented on issue #6602: request help: real_ip_recursive for real-ip plugin

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


   > 
   
   Am I missing something? But the logic here is just to return the last IP of XFF
   ![image](https://user-images.githubusercontent.com/21658981/158164111-a2d0b0d2-6bf6-484b-a4e6-c251ae71be61.png)
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] crazyMonkey1995 commented on issue #6602: request help: real_ip_recursive for real-ip plugin

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


   > > > 
   > > 
   > > 
   > > Am I missing something? But the logic here is just to return the last IP of XFF ![image](https://user-images.githubusercontent.com/21658981/158164111-a2d0b0d2-6bf6-484b-a4e6-c251ae71be61.png) @tokers
   > 
   > No, the current implementation of `real-ip` plugin is not totally same to the Nginx real-ip module. We may support the `real_ip_header` and `real_ip_recursive` settings.
   
   Ok, I misunderstood.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] tokers commented on issue #6602: request help: real_ip_recursive for real-ip plugin

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


   > > 
   > 
   > Am I missing something? But the logic here is just to return the last IP of XFF ![image](https://user-images.githubusercontent.com/21658981/158164111-a2d0b0d2-6bf6-484b-a4e6-c251ae71be61.png) @tokers
   
   No, the current implementation of `real-ip` plugin is not totally same to the Nginx real-ip module. We may support the `real_ip_header` and `real_ip_recursive` settings.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] tokers commented on issue #6602: request help: real_ip_recursive for real-ip plugin

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


   @crazyMonkey1995 Would you like to try it?


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] crazyMonkey1995 commented on issue #6602: request help: real_ip_recursive for real-ip plugin

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


   > @crazyMonkey1995 Would you like to try it?
   
   Okay, but maybe not so soon.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] tokers commented on issue #6602: request help: real_ip_recursive for real-ip plugin

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


   > > @crazyMonkey1995 Would you like to try it?
   > 
   > Okay, but maybe not so soon.
   
   Thanks!
   


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] tokers commented on issue #6602: request help: real_ip_recursive for real-ip plugin

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


   @crazyMonkey1995 I think it can be implemented in the real-ip plugin.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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



[GitHub] [apisix] crazyMonkey1995 edited a comment on issue #6602: request help: real_ip_recursive for real-ip plugin

Posted by GitBox <gi...@apache.org>.
crazyMonkey1995 edited a comment on issue #6602:
URL: https://github.com/apache/apisix/issues/6602#issuecomment-1066676047


   > 
   
   Am I missing something? But the logic here is just to return the last IP of XFF
   ![image](https://user-images.githubusercontent.com/21658981/158164111-a2d0b0d2-6bf6-484b-a4e6-c251ae71be61.png)
   @tokers 


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

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