You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by "FSchumacher (via GitHub)" <gi...@apache.org> on 2023/03/25 12:15:34 UTC
[GitHub] [tomcat] FSchumacher opened a new pull request, #604: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent
FSchumacher opened a new pull request, #604:
URL: https://github.com/apache/tomcat/pull/604
Give it its own log message instead of re-using the one from fireRequestInitEvent.
--
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: dev-unsubscribe@tomcat.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] [tomcat] markt-asf commented on a diff in pull request #604: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent
Posted by "markt-asf (via GitHub)" <gi...@apache.org>.
markt-asf commented on code in PR #604:
URL: https://github.com/apache/tomcat/pull/604#discussion_r1148503570
##########
webapps/docs/changelog.xml:
##########
@@ -174,6 +174,10 @@
with some OSGi custom URL schemes that can trigger potentially slow DNS
lookups in some configurations. (markt)
</fix>
+ <fix>
+ <bug>66543</bug>: Give `StandardContext#fireRequestDestroyEvent` its own
+ log message. (fschumacher)
Review Comment:
That formatting won't work here. You'll need to use `<code>...</code>`
--
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: dev-unsubscribe@tomcat.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[GitHub] [tomcat] markt-asf merged pull request #604: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent
Posted by "markt-asf (via GitHub)" <gi...@apache.org>.
markt-asf merged PR #604:
URL: https://github.com/apache/tomcat/pull/604
--
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: dev-unsubscribe@tomcat.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org