Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#1164 closed defect (fixed)

Changing outproxy list in i2ptunnel

Reported by: zzz Owned by: dg
Priority: minor Milestone: 0.9.13
Component: apps/i2ptunnel Version: 0.9.9
Keywords: Cc:
Parent Tickets: Sensitive: no

Description

In the outproxy error page we claim you can edit your outproxy list, but it's set in the I2PTunnelHTTPClient constructor and so won't take effect until restart.

To fix, override optionsUpdated() in I2PTunnelHTTPClientBase and fixup _proxyList - see I2PTunnelClient.optionsUpdated() for example.

Low priority since nobody updates their outproxy list.

Subtickets

Change History (4)

comment:1 Changed 6 years ago by dg

Owner: set to dg
Status: newassigned

I'll take a look at this soon. Assigning to me so I don't end up bumping into anybody else.

comment:2 Changed 5 years ago by zzz

Milestone: 0.9.120.9.13

status:

<dg> zzz: http://pastethis.i2p/show/DMpyslKx5RQ12UYbY5OK/ - this doesn't work right now, no effect.
<dg> I've had this since 0.9.10, I think but I haven't had time to really get in my head in and figure it out
<dg> I'd like to get it in but I really don't see what could be wrong with it
<dg> it follows the hint given (optionsUpdated) and it's simple

comment:3 Changed 5 years ago by zzz

Here's my stab at it, starting from where you left off. You were missing the part in TunnelController?. Untested.

http://pastethis.i2p/show/MLmMno8uS2bBrnQ46eqa/

comment:4 Changed 5 years ago by zzz

Resolution: fixed
Status: assignedclosed

Fixed as above in b105fedb29c78505ccb10e59f5e4aaf0be81c0b6 0.9.12-2

Last edited 5 years ago by zzz (previous) (diff)
Note: See TracTickets for help on using tickets.