I2P Address: [http://git.idk.i2p]

Skip to content
Snippets Groups Projects

i2ptunnel: Prep for keepalive

Merged zzz requested to merge zzz/i2p.i2p:i2ptunnel-keepalive-util into master

First part of keepalive changes.

  • Add utilities to find end-of-message
  • Move GunzipOutputStream to new util package
  • Add end-of-message checking to GunzipOutputStream
  • All new utilities unused in this checkin

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • zzz changed milestone to %2.5.0

    changed milestone to %2.5.0

  • zzz added i2ptunnel label

    added i2ptunnel label

  • zzz requested review from @idk

    requested review from @idk

  • zzz assigned to @zzz

    assigned to @zzz

  • Author Maintainer

    Details:

    To implement keepalive, the client must find the end of the response from the server, so it knows it can send the next request. There's 3 cases:

    • known size (content-length)
    • chunked
    • x-i2p-gzip

    This MR adds classes for the first two that do that, and adds support for it to the existing gunzipper class. The new classes are in a new util/ dir and the gunzipper is moved there for consistency.

    This MR does not use the new classes or the support added to gunzipper. That will happen in followup MRs.

  • idk approved this merge request

    approved this merge request

  • Owner

    Ack, thanks for the details, sorry for the long wait time on the review. Please merge when you're ready.

  • zzz merged

    merged

  • zzz mentioned in commit 37791e83

    mentioned in commit 37791e83

  • zzz mentioned in merge request !176 (merged)

    mentioned in merge request !176 (merged)

Please register or sign in to reply
Loading