Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove msgio double wrap #1065

Closed
wants to merge 1 commit into from
Closed

remove msgio double wrap #1065

wants to merge 1 commit into from

Commits on Apr 12, 2015

  1. remove msgio double wrap 

    There was doublewrapping with an unneeded msgio. given that we
use a stream muxer now, msgio is only needed by secureConn -- to
signal the boundaries of an encrypted / mac-ed ciphertext.

Side note: i think including the varint length in the clear is
actually a bad idea that can be exploited by an attacker. it should
be encrypted, too. (TODO)
    @jbenet
    jbenet committed Apr 12, 2015
    Configuration menu
    Copy the full SHA
    0fb7d25 View commit details
    Browse the repository at this point in the history