Opened 3 years ago

Closed 3 years ago

#1741 closed defect (fixed)

Change Jetty request logging from b64 to b32

Reported by: zzz Owned by: zzz
Priority: minor Milestone: 0.9.24
Component: apps/jetty Version: 0.9.23
Keywords: Cc:
Parent Tickets:

Description

Back when we started the custom I2P version of Jetty's NCSARequestLog, we didn't have the b32 naming system defined, and we logged the hash of the requestor's destination in b64. Now we have b32, and our typographical standard, so to speak, is that router hashes are displayed in b64 and destination hashes are displayed in 52chars.b32.i2p.

Also, a b64 hash isn't a legal hostname, it contains no dots, and contains '='. So let's log the b32 instead. Needs to change in net.i2p.jetty.I2PRequestLog.java and we must add an option to pick the old b64 if desired. That option would be in jetty.xml and we'll have to add a setB64() method. Which way to default it? New installs only or?

Subtickets

Change History (2)

comment:1 Changed 3 years ago by zzz

  • Owner set to zzz
  • Status changed from new to accepted

comment:2 Changed 3 years ago by zzz

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

Fixed in 0.9.24.

Note: See TracTickets for help on using tickets.