How to actually do a POST to change level?

[MikaelAnderssonWigander]

Hi

I'm testing this extension out but I can't find any documentation on how to actually POST a change of logging level.

Using httpie in CLI

 http --form POST :9000/q/logging-manager loggerName='com.myapp' loggerLevel='INFO' 

returns an error

{
    "details": "Error id f65b375d-4da4-44ed-93f4-f9186a14479f-12, java.lang.IllegalStateException: Request has already been read",
    "stack": "java.lang.IllegalStateException: Request has already been read\n\tat io.vertx.core.http.impl.Http1xServerRequest.checkEnded(Http1xServerRequest.java:733)\n\tat io.vertx.core.http.impl.Http1xServerRequest.setExpectMultipart(Http1xServerRequest.java:522)\n\tat io.vertx.core.http.impl.HttpServerRequestWrapper.setExpectMultipart(HttpServerRequestWrapper.java:190)\n\tat io.vertx.core.http.impl.HttpServerRequestWrapper.setExpectMultipart(HttpServerRequestWrapper.java:190)\n\tat io.vertx.ext.web.handler.impl.BodyHandlerImpl$BHandler.<init>(BodyHandlerImpl.java:230)\n\tat io.vertx.ext.web.handler.impl.BodyHandlerImpl.handle(BodyHandlerImpl.java:126)\n\tat io.vertx.ext.web.handler.impl.BodyHandlerImpl.handle(BodyHandlerImpl.java:45)\n\tat io.quarkus.vertx.http.runtime.VertxHttpRecorder$18.handle(VertxHttpRecorder.java:1625)\n\tat io.quarkus.vertx.http.runtime.VertxHttpRecorder$18.handle(VertxHttpRecorder.java:1596)\n\tat io.vertx.ext.web.impl.RouteState.handleContext(RouteState.java:1281)\n\tat io.vertx.ext.web.impl.RoutingContextImplBase.iterateNext(RoutingContextImplBase.java:177)\n\tat io.vertx.ext.web.impl.RoutingContextImpl.next(RoutingContextImpl.java:151)\n\tat io.quarkus.vertx.http.runtime.devmode.VertxHttpHotReplacementSetup$5.handle(VertxHttpHotReplacementSetup.java:206)\n\tat io.quarkus.vertx.http.runtime.devmode.VertxHttpHotReplacementSetup$5.handle(VertxHttpHotReplacementSetup.java:194)\n\tat io.vertx.core.impl.future.FutureImpl$4.onSuccess(FutureImpl.java:176)\n\tat io.vertx.core.impl.future.FutureBase.lambda$emitSuccess$0(FutureBase.java:60)\n\tat io.netty.util.concurrent.AbstractEventExecutor.runTask(AbstractEventExecutor.java:173)\n\tat io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:166)\n\tat io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:472)\n\tat io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:569)\n\tat io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:998)\n\tat io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)\n\tat io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)\n\tat java.base/java.lang.Thread.run(Thread.java:1583)"
}

Quarkus 3.19.2 and extension 3.3.2

Not sure if this is extension related or not…

[MikaelAnderssonWigander]

When querying for logs it returns fine:

http :9000/q/logging-manager loggerName==com.myapp
HTTP/1.1 200 OK
Content-Type: application/json
content-length: 65

{
    "configuredLevel": "DEBUG",
    "effectiveLevel": "DEBUG",
    "name": "com.myapp"
}

[phillip-kruger]

You can also set the log level (in dev mode) in Dev UI and the terminal . See https://www.youtube.com/live/1PJNbX8eWG0?feature=shared&t=1575

Try:

http --form POST http://localhost:8080/logs LOGGER_NAME_PARAM=myLogger LOGGER_LEVEL_PARAM=DEBUG Content-Type:application/x-www-form-urlencoded

[MikaelAnderssonWigander]

same response, doesn't work.
We use 9000 for the management port and if I switch to 8080 I get the index.html back

[phillip-kruger]

OK I meant you should use :

http --form POST http://localhost:9000/q/logging-manager LOGGER_NAME_PARAM=myLogger LOGGER_LEVEL_PARAM=DEBUG Content-Type:application/x-www-form-urlencoded

The important part is Content-Type:application/x-www-form-urlencoded

[MikaelAnderssonWigander]

http --verbose  --form POST :9000/q/logging-manager LOGGER_NAME_PARAM=dts LOGGER_LEVEL_PARAM=DEBUG 'Content-Type:application/x-www-form-urlencoded'

POST /q/logging-manager HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
Content-Length: 46
Content-Type: application/x-www-form-urlencoded
Host: localhost:9000
User-Agent: HTTPie/3.2.4

LOGGER_NAME_PARAM=dts&LOGGER_LEVEL_PARAM=DEBUG


HTTP/1.1 500 Internal Server Error
content-length: 2397
content-type: application/json; charset=utf-8

{
    "details": "Error id c7b7fde2-6696-4239-abb8-3928670bdfc5-6, java.lang.IllegalStateException: Request has already been read",
    "stack": "java.lang.IllegalStateException: Request has already been read\n\tat io.vertx.core.http.impl.Http1xServerRequest.checkEnded(Http1xServerRequest.java:733)\n\tat io.vertx.core.http.impl.Http1xServerRequest.setExpectMultipart(Http1xServerRequest.java:522)\n\tat io.vertx.core.http.impl.HttpServerRequestWrapper.setExpectMultipart(HttpServerRequestWrapper.java:190)\n\tat io.vertx.core.http.impl.HttpServerRequestWrapper.setExpectMultipart(HttpServerRequestWrapper.java:190)\n\tat io.vertx.ext.web.handler.impl.BodyHandlerImpl$BHandler.<init>(BodyHandlerImpl.java:230)\n\tat io.vertx.ext.web.handler.impl.BodyHandlerImpl.handle(BodyHandlerImpl.java:126)\n\tat io.vertx.ext.web.handler.impl.BodyHandlerImpl.handle(BodyHandlerImpl.java:45)\n\tat io.quarkus.vertx.http.runtime.VertxHttpRecorder$18.handle(VertxHttpRecorder.java:1625)\n\tat io.quarkus.vertx.http.runtime.VertxHttpRecorder$18.handle(VertxHttpRecorder.java:1596)\n\tat io.vertx.ext.web.impl.RouteState.handleContext(RouteState.java:1281)\n\tat io.vertx.ext.web.impl.RoutingContextImplBase.iterateNext(RoutingContextImplBase.java:177)\n\tat io.vertx.ext.web.impl.RoutingContextImpl.next(RoutingContextImpl.java:151)\n\tat io.quarkus.vertx.http.runtime.devmode.VertxHttpHotReplacementSetup$5.handle(VertxHttpHotReplacementSetup.java:206)\n\tat io.quarkus.vertx.http.runtime.devmode.VertxHttpHotReplacementSetup$5.handle(VertxHttpHotReplacementSetup.java:194)\n\tat io.vertx.core.impl.future.FutureImpl$4.onSuccess(FutureImpl.java:176)\n\tat io.vertx.core.impl.future.FutureBase.lambda$emitSuccess$0(FutureBase.java:60)\n\tat io.netty.util.concurrent.AbstractEventExecutor.runTask(AbstractEventExecutor.java:173)\n\tat io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:166)\n\tat io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:472)\n\tat io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:569)\n\tat io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:998)\n\tat io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)\n\tat io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)\n\tat java.base/java.lang.Thread.run(Thread.java:1583)"
}

[MikaelAnderssonWigander]

This is the code that gets executed?

private void handlePost(HttpServerRequest request, HttpServerResponse response) {
        String contentType = request.getHeader("Content-Type");
        if (!"application/x-www-form-urlencoded".equals(contentType)) {
            response.setStatusCode(415).end();
            return;
        }

        String loggerName = request.getFormAttribute(LOGGER_NAME_PARAM);
        String loggerLevel = request.getFormAttribute(LOGGER_LEVEL_PARAM);

        if (loggerLevel == null || loggerLevel.isEmpty()) {
            LogController.updateLogLevel(loggerName, null);
        } else {
            LogController.updateLogLevel(loggerName, loggerLevel);
        }

        response.setStatusCode(201).end();
    }

Using the actual string name of the field instead of the constant. Same same

[phillip-kruger]

Yes this is the code that should do it

[phillip-kruger]

Have you tried doing this in Swagger UI: https://github.com/quarkiverse/quarkus-logging-manager?tab=readme-ov-file#openapi

If it does not work there, it;s a bug.

Also try switching off the management port and see if if works in 8080. Then we can narrow it down to management port.

[MikaelAnderssonWigander]

I have no Swagger interface with the parameter set.
Nor can I change logging level from dev UI…

[MikaelAnderssonWigander]

I also removed management port and now I get Error 404

[MikaelAnderssonWigander]

I was at fault here, one more property needed to be false: quarkus.management.enable=false
Now it works on port 8080

[phillip-kruger]

OK sorry I could not help. This is probably a bug (I see there is no test cases for POST). THis might be a regression. Can you create an issue with a reproducer ? That will ensure a quick fix. Thanks

[MikaelAnderssonWigander]

There is a bug in the way when quarkus.management is enabled. It defaults to port 9000

[melloware]

@MikaelAnderssonWigander yes please create a reproducer and a bug and we can fix it.

[phillip-kruger]

I think management support came long after this extension, and we never added management support here... Should be an easy enough fix. A reproducer will help. Thanks

[MikaelAnderssonWigander]

Here you are

https://github.com/MikaelAnderssonWigander/logging-manager-management.git

[MikaelAnderssonWigander]

Issue created as well, #359