necouchman commented on a change in pull request #649:
URL: https://github.com/apache/guacamole-client/pull/649#discussion_r735199667
##########
File path:
guacamole/src/main/java/org/apache/guacamole/rest/RESTExceptionMapper.java
##########
@@ -50,37 +50,20 @@
private final Logger logger =
LoggerFactory.getLogger(RESTExceptionMapper.class);
/**
- * The HttpServletRequest for the Throwable being intercepted. Despite
this
- * class being a Singleton, this object will always be scoped with the
- * current request for the Throwable that is being processed by this class.
+ * The ContainerRequest for HTTP request that resulted in the Throwable
Review comment:
The wording of this line sounds a little off to me. If it's what you
intended, that's fine - it could just be my misunderstanding - but it looks
like either missing words or extra words, and I can't really tell which...
`The ContainerRequest for the HTTP request that resulted in...`
`The ContainerRequest that resulted in...`
Or I'm just missing something?
--
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]
