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

Implement external account binding (WIP) #21

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

paulgb
Copy link

@paulgb paulgb commented Jun 7, 2022

This PR implements external account binding as specified in RFC 8555 and adds a test for it.

The functionality is all there, but I'm marking this as WIP as there are a few things I'd like to clean up (tests and error handling). If the general approach is acceptable, I'm happy to complete those tasks before merging.

@paulgb
Copy link
Author

paulgb commented Jun 7, 2022

In addition to Pebble, I have successfully used this code for EAB ACME requests with ZeroSSL and Google's public CA.

@plaidfinch
Copy link

I would love to use this in a project I'm working on, and it would be great not to have to depend on the forked crate if possible. Can you give more details about what's needed to push this over the finish line and get it merged? Happy to help out!

@paulgb
Copy link
Author

paulgb commented Oct 20, 2022

@plaidfinch I know you don't want to rely on the forked crate, but FYI it is in cargo as acme2-eab and we use it in some things, so we intend to maintain it there until it is merged upstream (if it ever is).

@plaidfinch
Copy link

I'm happy to rely on the forked crate if it's maintained, I just don't want to end up depending on something that will drift out of sync with upstream. So if you're good with maintaining it, I'm happy to use that!

@paulgb
Copy link
Author

paulgb commented Oct 20, 2022

Yes, we use it in plane so we're happy to maintain the fork. I can't promise that we'll implement other new features, but we'll merge down upstream changes if any happen.

@plaidfinch
Copy link

@paulgb Great to hear, thanks for your work on this!

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.

None yet

2 participants