Github user mike-jumper commented on a diff in the pull request:

    https://github.com/apache/guacamole-client/pull/291#discussion_r191607392
  
    --- Diff: 
guacamole/src/main/java/org/apache/guacamole/rest/RESTServiceModule.java ---
    @@ -83,11 +83,9 @@ protected void configureServlets() {
             bind(AuthenticationService.class);
             
bind(AuthTokenGenerator.class).to(SecureRandomAuthTokenGenerator.class);
             bind(DecorationService.class);
    -
    -        // Automatically translate GuacamoleExceptions for REST methods
    -        MethodInterceptor interceptor = new RESTExceptionWrapper();
    -        requestInjection(interceptor);
    -        bindInterceptor(Matchers.any(), new RESTMethodMatcher(), 
interceptor);
    +        
    +        // Get the ExceptionMapper that will rewrite exceptions into JSON.
    --- End diff --
    
    `bind()` is not "getting" the `ExceptionMapper`. In this case, the original 
comment is probably still more appropriate.


---

Reply via email to