Changeset 5ede0a1 for router


Ignore:
Timestamp:
Jun 9, 2019 11:48:03 AM (10 months ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
0c54b6d
Parents:
2c97dc2
Message:

UPnP: Set lease duration of 3 hours, always refresh the lease

File:
1 edited

Legend:

Unmodified
Added
Removed
  • router/java/src/net/i2p/router/transport/UPnP.java

    r2c97dc2 r5ede0a1  
    911911                add.setArgumentValue("NewPortMappingDescription", description);
    912912                add.setArgumentValue("NewEnabled","1");
    913                 add.setArgumentValue("NewLeaseDuration", 0);
     913                // 3 hours
     914                // MUST be longer than max RI republish which is 52 minutes
     915                add.setArgumentValue("NewLeaseDuration", 3*60*60);
    914916               
    915917                boolean rv = add.postControlAction();
     
    10861088                                // Ports in portsToForwardNow but not in ports we must dump
    10871089                                for(ForwardPort port: ports) {
     1090                                        // Always add, since we now have a 3 hour lease duration,
     1091                                        // so we have to keep refreshing the lease.
    10881092                                        //if(portsToForward.contains(port)) {
    10891093                                        // If not in portsForwarded, it wasn't successful, try again
    1090                                         if(portsForwarded.contains(port)) {
     1094                                        //if(portsForwarded.contains(port)) {
    10911095                                                // We have forwarded it, and it should be forwarded, cool.
    10921096                                                // Big problem here, if firewall resets, we don't know it.
    10931097                                                // Do we need to re-forward anyway? or poll the router?
    1094                                         } else {
     1098                                        //} else {
    10951099                                                // Needs forwarding
    10961100                                                if(portsToForwardNow == null) portsToForwardNow = new HashSet<ForwardPort>();
    10971101                                                portsToForwardNow.add(port);
    1098                                         }
     1102                                        //}
    10991103                                }
    11001104                                for(ForwardPort port : portsToForward) {
Note: See TracChangeset for help on using the changeset viewer.