You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/11/07 18:15:43 UTC

[GitHub] [trafficserver] bneradt opened a new pull request, #9182: per_server.connection.max: CONNECT requests

bneradt opened a new pull request, #9182:
URL: https://github.com/apache/trafficserver/pull/9182

   We were putting servers that hit the per_server.connection.max limit on the dead server list if the client were connecting to them via HTTP CONNECT requests. This fixes the handling of these so the server is not put on the dead server list.


-- 
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] SolidWallOfCode commented on pull request #9182: per_server.connection.max: CONNECT requests

Posted by GitBox <gi...@apache.org>.
SolidWallOfCode commented on PR #9182:
URL: https://github.com/apache/trafficserver/pull/9182#issuecomment-1306297066

   I wonder if `EUSERS` should be propagated back and checked at a higher level, but that likely gets ugly.


-- 
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 #9182: per_server.connection.max: CONNECT requests

Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #9182:
URL: https://github.com/apache/trafficserver/pull/9182#issuecomment-1306023221

   My branch testing shows that the issue this PR fixes was introduced via the hostdb restructure PR (#8953). Since that did not go into 9.2.x or earlier, this fix is only needed for master and 10-Dev and should not be cherry-picked elsewhere.


-- 
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 merged pull request #9182: per_server.connection.max: CONNECT requests

Posted by GitBox <gi...@apache.org>.
bneradt merged PR #9182:
URL: https://github.com/apache/trafficserver/pull/9182


-- 
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 #9182: per_server.connection.max: CONNECT requests

Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #9182:
URL: https://github.com/apache/trafficserver/pull/9182#issuecomment-1306132655

   [approve ci autest]


-- 
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 #9182: per_server.connection.max: CONNECT requests

Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #9182:
URL: https://github.com/apache/trafficserver/pull/9182#issuecomment-1306073232

   [approve ci autest]


-- 
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 #9182: per_server.connection.max: CONNECT requests

Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #9182:
URL: https://github.com/apache/trafficserver/pull/9182#issuecomment-1306141557

   [approve ci autest]


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