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.

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

RFC 5544 Implementation #1258

Merged
merged 5 commits into from
Jan 8, 2024
Merged

RFC 5544 Implementation #1258

merged 5 commits into from
Jan 8, 2024

Conversation

enri1196
Copy link
Contributor

I've included all the structures for the RFC, there are still some problems with mixed encodings.

For reference: https://www.rfc-editor.org/rfc/rfc5544

@enri1196 enri1196 mentioned this pull request Nov 16, 2023
8 tasks
@leotaku
Copy link

leotaku commented Nov 19, 2023

Thanks for pinging me! Could you maybe describe to me/give a code example of the "problems with mixed encoding"? Without the full context, it is quite hard for me to figure out the issue.

Also, unrelated, but I assume that if this format gets accepted it should be a new crate rather than part of cms?

@baloo
Copy link
Member

baloo commented Nov 19, 2023

I don't think I've seen it used except for CMS. I think it's fine there.

cms/src/timestamped_data.rs Outdated Show resolved Hide resolved
enri1196 and others added 2 commits November 19, 2023 12:16
@baloo
Copy link
Member

baloo commented Nov 19, 2023

@enri1196 you wouldn't happen to be working on PE signatures by any chance? (if so, nixos folks are bringing up their own tooling to do it and this should be one of the pieces we were missing)

@enri1196
Copy link
Contributor Author

@baloo No, I did not know about the project from nixos folks. I’m happy to know that this PR can help them, can you show me something about it?

@leotaku I’m still investigating this problem ‘cause I’m still not sure if I’m the one doing things wrong or there’s a problem with the way some of the documents at my current job are generated. Let me know if for your use case everything is alright. Thanks in advance.

@baloo
Copy link
Member

baloo commented Nov 20, 2023

Most of it happens on matrix: https://matrix.to/#/#secure-boot:nixos.org
but otherwise:

@baloo
Copy link
Member

baloo commented Nov 20, 2023

This looks good to me, but I'd like to run this against goblin before merging that (I think it will require a couple of AssociatedOid)

@baloo baloo self-assigned this Nov 28, 2023
@enri1196
Copy link
Contributor Author

enri1196 commented Jan 8, 2024

any news?

@baloo
Copy link
Member

baloo commented Jan 8, 2024

I didn't have time to integrate that with a builder quite yet. But I don't see much of an issue with merging this.
I will want to revisit at a later date to add the builder side though.

@tarcieri tarcieri merged commit 562244a into RustCrypto:master Jan 8, 2024
17 checks passed
@tarcieri tarcieri mentioned this pull request Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants