Skip to content
This repository has been archived by the owner on Sep 6, 2022. It is now read-only.

Record envelope protobuf does not match spec #160

Closed
jacobheun opened this issue Jun 22, 2020 · 2 comments
Closed

Record envelope protobuf does not match spec #160

jacobheun opened this issue Jun 22, 2020 · 2 comments
Labels
kind/bug A bug in existing code (including security flaws)

Comments

@jacobheun
Copy link

Signature index here is set to 5, https://github.com/libp2p/go-libp2p-core/blob/v0.6.0/record/pb/envelope.proto#L29.

The spec is 4, which is should be. https://github.com/libp2p/specs/pull/217/files#diff-1d3e44bb1e34e0581019f35b96151a0dR70

@jacobheun jacobheun added the kind/bug A bug in existing code (including security flaws) label Jun 22, 2020
@aschmahmann
Copy link
Contributor

@jacobheun is this issue closed since the spec was changed to match?

@jacobheun
Copy link
Author

Yep, closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
Development

No branches or pull requests

2 participants