Paulo Lopes

Results 282 comments of Paulo Lopes

@muhammad-abubakar, I'm not 100% sure about having individual exceptions for each handler: Say we have: * `CorsValidationException` * `JWTAuthorizationException` * `Oauth2AuthorizationException` * `HttpForbiddenException` What would be the real difference between...

@muhammad-abubakar what is your oppinion on this style: https://github.com/vert-x3/vertx-web/blob/issues/2002/vertx-web/src/main/java/io/vertx/ext/web/handler/HttpException.java If you import the static method, you could use it like this: ```java switch (httpStatusCodeOf(exception)) { case 401: ((HttpException) exception) .catchFrom(CSRFHandler.class,...

@aomsweet the current API, allows something like: ```java httpRequest.authentication(new UsernamePasswordCredentials("username", "password")); ``` I think we should have a second method: ```java httpRequest.proxyAuthentication(new UsernamePasswordCredentials("username", "password")); ``` That uses: `Proxy-Authorization` as the...

@smwg the description isn't enough to troubleshoot the problem. The example you wrote is used often in our test suite and seems to be valid. Can you provide a small...

This is a graaljs issue: https://github.com/graalvm/graaljs/issues/37 still waiting for it to be fixed upstream.

This is a regular log message, all can be controlled using the desired logger config. You can read about it on the manual https://vertx.io/docs/vertx-core/java/#_logging

@narras-oss what would you suggest? Log as warning?

Ok, let's plan like this. This is part of a larger work where we improve the exception hierarchy in web. While you think normalization is not essential to be logged,...

The javadoc does mention that it does return a new instance. The reason behind that change was mostly to ensure that openapi would work as expected but it seems that...

@paulswithers please, if you can provide a PR updating the docs + examples in the docs it would be really nice! Thanks!