Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323497 - Add Import-Package ch.qos.logback.access.jetty.v7 to bundle org.eclipse.jetty.osgi.boot.logback
Summary: Add Import-Package ch.qos.logback.access.jetty.v7 to bundle org.eclipse.jetty...
Status: CLOSED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: osgi (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 7.1.x   Edit
Assignee: Hugues Malphettes CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-24 10:10 EDT by Christian Brensing CLA
Modified: 2011-09-20 15:52 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Brensing CLA 2010-08-24 10:10:47 EDT 
Build Identifier: 

The fragment bundle org.eclipse.jetty.osgi.boot.logback injects the logback dependencies into the Boot-Bundle. That works fine with LOGBack-Classic. In order to use LOGBack-Access as an alternative to the Jetty-Builtin-Request-Logger, the package ch.qos.logback.access.jetty.v7 must be (optionally) injected as well, otherwise a CNFE will be thrown.

jetty.xml example:

<Ref id="RequestLog">
  <Set name="requestLog">
    <New id="RequestLogImpl" class="ch.qos.logback.access.jetty.v7.RequestLogImpl"/>
  </Set>
</Ref>

It would be nice, if the package ch.qos.logback.access.jetty.v7 could be added as an optional import to the Import-Package list:

Import-Package: ..., ch.qos.logback.access.jetty.v7;resolution:=optional

Reproducible: Always
Comment 1 Hugues Malphettes CLA 2010-08-24 11:21:43 EDT 
Thanks Christian for reporting this.
I have committed the suggested change for jetty-7.2 and jetty-7.1
Comment 2 Jesse McConnell CLA 2011-09-20 15:52:10 EDT 
Resolved -> Closed