#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 |
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)
Change History (5)
Changed 7 years ago by
comment:1 Changed 7 years ago by
- 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
Status: | new → testing |
---|
comment:3 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | testing → closed |
Pushed in rev af7c845be25ea12beb088402474476862c7d28c7
Thanks wockenfuss!