Opened 9 months ago

Closed 9 months ago

Last modified 9 months ago

#2225 closed defect (fixed)

Clicking "Generate" Button on New Hidden Service Server Tunnel page creates 6 identical tunnels

Reported by: slumlord Owned by: zzz
Priority: minor Milestone: 0.9.35
Component: apps/i2ptunnel Version: 0.9.34
Keywords: hidden service manager, encrypt leaseset Cc: zzz
Parent Tickets:

Description

Steps to reproduce:

  1. Open I2P Tunnel Manager Page - http://localhost:7657/i2ptunnelmgr
  2. Click Create under I2P Hidden Services section - http://localhost:7657/i2ptunnel/edit?type=server
  3. Click Generate in Encrypt Leaseset section

Expected Result:

A new key is generated for leaseset encryption

Actual Result:

Returned to http://localhost:8660/i2ptunnelmgr page with 6 identical tunnels.

Running on:

I2P version:	0.9.34-9
Java version:	Oracle Corporation 1.8.0_151 (OpenJDK Runtime Environment 1.8.0_151-b12)
Wrapper version:	3.5.34
Server version:	9.2.24.v20180105
Servlet version:	Jasper JSP 2.3 Engine
JSTL version:	standard-taglib 1.2.0
Platform:	Linux amd64
Jcpuid version:	3
Processor:	Ivy Bridge (coreisbr)
Jbigi:	Locally optimized library libjbigi-linux-coreisbr_64.so loaded from file
Jbigi version:	3
GMP version:	6.0.0
Encoding:	UTF-8
Charset:	UTF-8
Built By:	BobTheBuilder.i2P

MTN revid: e8cb10fd5c553c5cb4ff1e163391048ed425e978

Subtickets

Change History (5)

comment:1 Changed 9 months ago by slumlord

Perhaps a note should be added that destination-key pairs are to be added on the http://localhost:7657/configkeyring page? It wasn't immediately clear that this was required. I was looking on the client tunnel config page.

comment:2 Changed 9 months ago by zzz

  • Component changed from apps/console to apps/i2ptunnel
  • Milestone changed from undecided to 0.9.35
  • Status changed from new to accepted

reproduced here

comment:3 Changed 9 months ago by zzz

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

In 5d74a84e3a9c73df1e083d319e73990446cfe89b 0.9.34-11

comment:4 Changed 9 months ago by slumlord

Thanks zzz. I have tested the fix and it does work.
I did want to clarify the expected behavior, is the intention for the encryption key to be created as well as the tunnel? My expectation when clicking the 'Generate' button would be for the encryption key to be generated, while leaving the user on the create tunnel page to continue making changes - with tunnel creation carried out only when the user clicks the 'Save' button.

comment:5 Changed 9 months ago by zzz

The edit page doesn't have a form handler so we have to send all changes back the list page, which creates the tunnl. It is a little wonky. Encryption keys for tunnels is long-deprecated, if we ever fix it up maybe we can make the form work better also.

Note: See TracTickets for help on using tickets.