[GitHub] [trafficserver] shinrich opened a new pull request #8191: Fix error connection logging crash

[GitBox <gi...@apache.org>]

shinrich opened a new pull request #8191:
URL: https://github.com/apache/trafficserver/pull/8191


   If unmapped_url is not set, the code that logs a connection failure will crash.  We had this reported by a customer using a plugin which meant no remap rule was traversed.  This fix addressed their issue.
   
   This code was brought in PR #7757, so any branch with that change also needs this change.


-- 
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: github-unsubscribe@trafficserver.apache.org

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

[GitHub] [trafficserver] shinrich merged pull request #8191: Fix error connection logging crash

[GitBox <gi...@apache.org>]

shinrich merged pull request #8191:
URL: https://github.com/apache/trafficserver/pull/8191


   


-- 
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: github-unsubscribe@trafficserver.apache.org

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

[GitHub] [trafficserver] bneradt commented on pull request #8191: Fix error connection logging crash

[GitBox <gi...@apache.org>]

bneradt commented on pull request #8191:
URL: https://github.com/apache/trafficserver/pull/8191#issuecomment-891092894


   Looks good to me.


-- 
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: github-unsubscribe@trafficserver.apache.org

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