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

encoding/asn1: support decoding BER-encoded messages (e.g. PKCS7) #12267

Closed
dtbartle opened this issue Aug 21, 2015 · 6 comments
Closed

encoding/asn1: support decoding BER-encoded messages (e.g. PKCS7) #12267

dtbartle opened this issue Aug 21, 2015 · 6 comments
Labels
FrozenDueToAge
Milestone
Unplanned

Comments

@dtbartle
Copy link

There's a couple blockers to being able to parse a BER-encoded PKCS7 message:

  1. Support indefinite-length encoding.
  2. Support "ANY DEFINED BY".
@dtbartle dtbartle changed the title encoding/asn1 should support decoding BER-encoded messages (e.g. PKCS7) encoding/asn1: support decoding BER-encoded messages (e.g. PKCS7) Aug 21, 2015
@gopherbot
Copy link
Contributor

CL https://golang.org/cl/13746 mentions this issue.

@mikioh
Copy link
Contributor

mikioh commented Aug 22, 2015

/CC @agl

Also please take a look at #8860 and #10621.

@agl
Copy link
Contributor

agl commented Aug 22, 2015

encoding/asn1's design hasn't worked out too well and I don't think that BER support would be suitable.

@agl agl closed this as completed Aug 22, 2015
@dtbartle
Copy link
Author

I'm aware of at least three other libraries that need to support BER encoding. Two of them ends up ~reimplementing all of asn1 (without nicer tagging/unmarshaling support), and the other tries to convert BER to DER but does it wrong. Afaict they could use golang's asn1 if it supported indefinite-length encodings. The change to implement this is relatively small, and would help reduce the proliferation of buggy implementations.

@mikioh mikioh added this to the Unplanned milestone Aug 23, 2015
@nathany
Copy link
Contributor

nathany commented Feb 3, 2016

David, since this issue was closed, what did you end up doing for PKCS 7 and BER instead?

@nathany nathany mentioned this issue Feb 3, 2016
Open
@dtbartle
Copy link
Author

dtbartle commented Feb 4, 2016

I ended up writing a "BER-to-DER" converter, which I'd like to open-source, but haven't had a chance to yet.

@minux minux mentioned this issue Feb 4, 2016
Closed
@golang golang locked and limited conversation to collaborators Feb 3, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Unplanned
Development

No branches or pull requests
5 participants
@nathany @agl @mikioh @dtbartle @gopherbot