Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 282327

Summary:

[tooling] provide log level for jetty

Product:

[RT] RAP

Reporter:

Benjamin Muskalla <b.muskalla>

Component:

Tools

Assignee:

Project Inbox <rap-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

enhancement

Priority:

Version:

1.3

Target Milestone:

1.3 M2

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
patch+testcase	none

Description Benjamin Muskalla

2009-07-02 18:22:38 EDT

We should provide a combo to select the server-side log level for the jetty engine.
This can be configured with the "org.eclipse.equinox.http.jetty.log.stderr.threshold" vm parameter.

Jetty will use a basic stderr logger if no other logging mechanism is provided.
This setting can be used to over-ride the stderr logger threshold(and only this default logger)
Valid values are in increasing threshold: "debug", "info", "warn", "error", and "off"
(default threshold is "warn")

Comment 1 Ralf Sternberg

2009-07-19 05:29:52 EDT

I wonder whether users really need that level of configuration for RAP development?
We could also add controls to enable HTTPS in jetty (which needs at least 3 controls...) and I guess there are more ideas.
I think we should focus on RAP-related configuration in the launcher.

If we add a parameter to set the log level to "info" by default - wouldn't that make almost everyone happy?

Comment 2 Benjamin Muskalla

2009-07-19 14:05:19 EDT

As long as an advanced user can still override the flag by specifying a VM argument I'm fine with just adding it in the background :)

Comment 3 Benjamin Muskalla

2009-08-09 08:50:49 EDT

Created attachment 143851 [details]
patch+testcase

Patch to add the log level "info" as default.
I wonder why we prefer values set by the launcer over custom values. Changed the order (see failing test).
In my eyes we should allow to override the values set by the launer (and changing the testcase accordingly)

Comment 4 Benjamin Muskalla

2009-08-19 06:41:19 EDT

We agreed to not switch the order but adding the default log level to new RAP launch configs. This we users are still able to override it without touching the launcher too much.

Comment 5 Rüdiger Herrmann

2009-09-11 10:09:01 EDT

Fixed in CVS HEAD