You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2021/03/02 16:58:04 UTC

[GitHub] [activemq] mattrpav opened a new pull request #615: AMQ-6951 Hide jetty version

mattrpav opened a new pull request #615:
URL: https://github.com/apache/activemq/pull/615


   Fixes:
   
   * Console
   * WS transport
   * HTTP transport


----------------------------------------------------------------
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] [activemq] jbonofre commented on pull request #615: AMQ-6951 Hide jetty version

Posted by GitBox <gi...@apache.org>.
jbonofre commented on pull request #615:
URL: https://github.com/apache/activemq/pull/615#issuecomment-801665930


   Hey @mattrpav . I guess the purpose is for security reason: to "hide" the Jetty server version for clients (and so avoid potential use of Jetty version header for a known attack). Correct ?


----------------------------------------------------------------
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] [activemq] mattrpav merged pull request #615: AMQ-6951 Hide jetty version

Posted by GitBox <gi...@apache.org>.
mattrpav merged pull request #615:
URL: https://github.com/apache/activemq/pull/615


   


----------------------------------------------------------------
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] [activemq] jbonofre commented on pull request #615: AMQ-6951 Hide jetty version

Posted by GitBox <gi...@apache.org>.
jbonofre commented on pull request #615:
URL: https://github.com/apache/activemq/pull/615#issuecomment-801666752


   OK I saw the discussion in the Jira, it makes sense. I don't think we need to do it configurable for the transport connector. I will test the PR rebased on `master` as I fixed the tests ;)


----------------------------------------------------------------
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] [activemq] lucastetreault removed a comment on pull request #615: AMQ-6951 Hide jetty version

Posted by GitBox <gi...@apache.org>.
lucastetreault removed a comment on pull request #615:
URL: https://github.com/apache/activemq/pull/615#issuecomment-792579893


   Can one of the admins verify this patch?


----------------------------------------------------------------
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] [activemq] lucastetreault commented on pull request #615: AMQ-6951 Hide jetty version

Posted by GitBox <gi...@apache.org>.
lucastetreault commented on pull request #615:
URL: https://github.com/apache/activemq/pull/615#issuecomment-792579893


   Can one of the admins verify this patch?


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