Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#827 closed defect (fixed)

Cannot change I2CP port on client side

Reported by: str4d Owned by: str4d
Priority: minor Milestone: 0.9.10
Component: apps/BOB Version: 0.9.4
Keywords: Cc:
Parent Tickets:

Description

I found this bug when trying to change the I2CP port that BOB-one.jar uses. I set i2cp.tcp.port=#### in bob.config (where #### is the local port of the SSH tunnel that is forwarding the I2CP port of the remote I2P router), but the I2CP client API only ever connects to port 7654. This was verified by changing the local port of the SSH tunnel to 7654 while leaving the config variable set to the original local port - BOB-one.jar is then able to successfully connect to the remote I2P router.

Subtickets

Change History (6)

comment:1 Changed 6 years ago by zzz

  • Component changed from api/i2cp to apps/BOB
  • Owner changed from zzz to sponge
  • Status changed from new to assigned

All the support should be working in I2CP, but I haven't tested it recently.

Assigning to sponge to verify that he's passing the bob config property through to the socket manager correctly.

comment:2 Changed 6 years ago by zzz

the other thing to check is if you have bob.config in the right place and it's picking it up at all.

comment:3 Changed 6 years ago by str4d

I am modifying the bob.config that is generated in the same dir as BOB-one.jar when none exists, so it is definitely the right one. And querying BOB-one while it is running confirms that it is picking up the Properties.

comment:4 Changed 6 years ago by str4d

  • Milestone 0.9.5 deleted

comment:5 Changed 5 years ago by str4d

  • Milestone set to 0.9.10
  • Resolution set to fixed
  • Status changed from assigned to closed

The problem was that I2PSocketManagerFactory.createManager(PKStream, opts) was being called with the I2CP settings in opts, but that method overwrites them with the System-defined settings. Calling I2PSocketManagerFactory.createManager(PKStream, i2cpHost, i2cpPort, opts) instead solved the problem.

Pushed in rev 2f9fa6e68bbb0d8ac3b06c90621d24431d692b73

comment:6 Changed 5 years ago by str4d

  • Owner changed from sponge to str4d
Note: See TracTickets for help on using tickets.