Opened 11 years ago

Closed 7 years ago

Last modified 7 years ago

#335 closed defect (fixed)

Leading null bytes received in HTTP Server

Reported by: zzz Owned by: zzz
Priority: minor Milestone: 0.9.19
Component: apps/i2ptunnel Version: 0.6.31
Keywords: Cc:
Parent Tickets: Sensitive: no


Connections sometimes start with a spurious single zero byte, and more rarely, more than one zero bytes.

Discovered in in early 2006; workaround implemented by jrandom in Still happening today as seen in the i2ptunnel.httpNullWorkaround stat. Unknown if the cause is on the client or server side, or if it is specific to HTTP or is in the I2CP or streaming layer. If the bug is not in the HTTP client-to-server direction, but is more general, then the workaround is not completely effective and it is a major bug.


Change History (9)

comment:1 Changed 11 years ago by sponge

Interesting bug. I gladly can beat to death the streaming lib and see if it happens there via BOB and post a new comment. That will at least rule out streaming lib as a problem. I've never seen this happen in a BOB created tunnel, so I don't believe it's in the streaming lib, but I'll verify it for you anyway.

comment:2 Changed 11 years ago by sponge

Been testing via Robert for weeks, and not seen this happen yet. I'm fairly certain that you can rule out streaminglib. The test will be part of the next beta release (in a short while from this post time) and will be incorporated into the next stable version. With many acting as testers, we'll get a report if the event happens.

comment:3 Changed 10 years ago by zzz

Owner: set to zzz
Status: newaccepted

comment:4 Changed 10 years ago by sponge

10 months, and not a single report of Robert spilling out the message, so, not streaming lib. That leaves the server and proxy sides.

comment:5 Changed 10 years ago by str4d


comment:6 Changed 9 years ago by zzz

Milestone: 0.8.14

Milestone 0.8.14 deleted

comment:7 Changed 7 years ago by zzz

Milestone: 0.9.19
Status: acceptedtesting

Possible fix in 064616c2027d7a6aff3bcefea141ac580cafea37 0.9.18-17-rc

comment:8 Changed 7 years ago by zzz

Resolution: fixed
Status: testingclosed

no nulls seen after several hours, declaring fixed.

comment:9 Changed 7 years ago by zzz

No errors seen after 4 days on a server that previously saw ~2 per hour. Removing workaround and stat in 77405a1f959e22f986ea38239259c6628e888131 i2p.i2p.zzz.test2 to be propped for 0.9.20.

Note: See TracTickets for help on using tickets.