Hide Maven from webapps running inside Jetty

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 6.1.5
  • Fix Version/s: 7.0.0pre0
  • Component/s: Maven
  • Labels:
    None
  • Patch Submitted:
    Yes

Description

When I run Jetty by using jetty:run, the webapp running inside Jetty gets to see the Maven classes and resources that surround maven-jetty-plugin.

This causes a problem for web applications that themselves depend on Plexus/Maven. It would be nice if Jetty can hide the surrounding Maven from web apps and provide sufficient isolation.

I'll attach a patch.

  1. File
    JETTY-463
    10/Nov/07 5:13 PM
    3 kB
    Kohsuke Kawaguchi
  2. Text File
    JETTY-463.patch
    18/Dec/07 12:59 PM
    1 kB
    Jesse McConnell

Issue Links

depends upon

Improvement - An improvement or enhancement to an existing feature or task. JETTY-1096 provide an option to exclude jetty/maven/plexus from the parent class loader when running web apps via "mvn jetty:run"

  • Major - Major loss of function.
  • Resolved - A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.
is related to

Improvement - An improvement or enhancement to an existing feature or task. JETTY-1096 provide an option to exclude jetty/maven/plexus from the parent class loader when running web apps via "mvn jetty:run"

  • Major - Major loss of function.
  • Resolved - A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Kohsuke Kawaguchi added a comment -

Patch.

Show
Kohsuke Kawaguchi added a comment - Patch.
Hide
Permalink
Greg Wilkins added a comment -

Jesse,

this should be able to be done simply by configuring the maven packages as server classes on the context.

Show
Greg Wilkins added a comment - Jesse, this should be able to be done simply by configuring the maven packages as server classes on the context.
Hide
Permalink
Jesse McConnell added a comment -

greg, patch attached..that what you were thinking?

Show
Jesse McConnell added a comment - greg, patch attached..that what you were thinking?
Hide
Permalink
Jan Bartel added a comment -

Patch was applied to jetty trunk as svn ref 2301.

Jan

Show
Jan Bartel added a comment - Patch was applied to jetty trunk as svn ref 2301. Jan

People

  • Assignee:
    Jesse McConnell
    Reporter:
    Kohsuke Kawaguchi
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: