Opened 7 years ago

Closed 7 years ago

Last modified 7 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: Sensitive: no


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.


Attachments (1)

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

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by wockenfuss

Attachment: 1165.diff added

comment:1 Changed 7 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 7 years ago by wockenfuss

Status: newtesting

comment:3 Changed 7 years ago by str4d

Resolution: fixed
Status: testingclosed

Pushed in rev af7c845be25ea12beb088402474476862c7d28c7

Thanks wockenfuss!

Note: See TracTickets for help on using tickets.