Opened 5 years ago

Closed 5 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:


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 5 years ago by zzz

  • Milestone changed from 0.9.15 to 0.9.12
  • Owner set to zzz
  • Status changed from new to accepted

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

comment:2 Changed 5 years ago by zzz

  • Component changed from router/update to www/i2p

propped in 0.9.11-2

leaving open for docs

comment:3 Changed 5 years ago by zzz

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

Docs updated in fcf66ca83d15f9afcb35b1f14abfb38779d43401

New page

Note: See TracTickets for help on using tickets.