Opened 7 years ago

Closed 7 years ago

#1185 closed task (fixed)

Convert UpdateManager to ClientApp

Reported by: zzz Owned by: zzz
Priority: maintenance Milestone: 0.9.12
Component: www/i2p Version: 0.9.10
Keywords: Cc:
Parent Tickets: Sensitive: no


0.9.4 brought both the UpdateManager? and the ClientAppManager?, with hooks in Context to find them. But ClientAppManager? has a general-purpose way to find applications ("registration"), which is now used by the HTTP client to find the Orchid plugin.

So the correct way for i2psnark to find the ClientAppManager? is identical, have the RouterAppManager? just register, and remove the UpdateManager? hooks in the Context.

This should be OK. The ClientAppManager? doesn't shutdown registered apps, does it? (no, just started apps, it looks like)

We don't want the ClientAppManager? to start the UpdateManager?… just have the UpdateManager? register with it. (just like Orchid).

Also we should really document the ClientAppManager? and PortManager? on the website.

Somehow this didn't occur to me in 0.9.4.

Low priority, this is just a cleanup.


Change History (3)

comment:1 Changed 7 years ago by zzz

Owner: set to zzz
Status: newaccepted

In i2p.i2p.zzz.test2 911a8782667d878dd7a2976c7fb10d481d80e023 to be propped for 0.9.12

comment:2 Changed 7 years ago by zzz

Component: router/updatewww/i2p

propped in 0.9.11-2

leaving open for docs

comment:3 Changed 7 years ago by zzz

Resolution: fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.