shukitchan commented on PR #12510:
URL: https://github.com/apache/trafficserver/pull/12510#issuecomment-3314180540

   On second thought, perhaps we should just take care of the rest 
consistently. 
   
   may be we should fix up `ts_lua_server_request.cc` as well. 
   - There are places where we are retrieving the ip address or port of the 
next destination. Even though they are not going to be AF_UNIX, for 
completeness sake, we should just take care of handling only for 
AF_INET/AF_INET6 and use the default for anything else.
   - There are also places where we can `set` address in this file. It makes no 
sense for anything other than INET/INET6. So we can just error out gracefully 
in those places as well if invalid values are passed. 
   
   Thanks so much for your patience and help.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to