Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#1165 closed task (fixed)

Remove remaining Jetty dependencies in susimail

Reported by: zzz Owned by:
Priority: maintenance Milestone: 0.9.12
Component: apps/susimail Version: 0.9.9
Keywords: Cc:
Parent Tickets:

Description

Our webapps should depend only on javax.servlet, not Jetty classes. Jetty dependencies cause trouble when we update Jetty, and prevent people from running the webapps under other web servers.

I2psnark dependencies on Jetty were removed in 2013 at great effort. Other than routerconsole, which must depend on Jetty since it starts it, the only webapp with remaining dependencies is susimail.

Most is already done; form processing classes (MultiPartRequest? et al) were copied out of Jetty 6 and bundled with susimail when we moved to Jetty 7.

What remains (search for 'eclipse') is only MultiMap?. Unfortunately MultiMap? requires other Jetty utils so I punted in the port to Jetty 7 and left it in.

Either write a compatible MultiMap? (preferred) or copy the required sources in to susimail.

Subtickets

Attachments (1)

1165.diff (1.2 KB) - added by wockenfuss 5 years ago.

Download all attachments as: .zip

Change History (5)

Changed 5 years ago by wockenfuss

comment:1 Changed 5 years ago by wockenfuss

  • reimplemented needed MultiMap? functionality in local class
  • file upload tested and working
  • only depends from i2p.jar and javax.servlet.jar now

comment:2 Changed 5 years ago by wockenfuss

  • Status changed from new to testing

comment:3 Changed 5 years ago by str4d

  • Resolution set to fixed
  • Status changed from testing to closed

Pushed in rev af7c845be25ea12beb088402474476862c7d28c7

Thanks wockenfuss!

comment:4 Changed 5 years ago by zzz

fix up build.xml in 5afe57cebc4f97706dabceeaca229c78666ee003

Note: See TracTickets for help on using tickets.