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

Next steps for maintenance / development of this crate 🚀 #40

Open
dunxen opened this issue Nov 16, 2020 · 10 comments
Open

Next steps for maintenance / development of this crate 🚀 #40

dunxen opened this issue Nov 16, 2020 · 10 comments

Comments

@dunxen
Copy link
Member

dunxen commented Nov 16, 2020

This is a tracking issue for discussion around the future direction of this crate. 😄

The original crate has been yanked and the macaroon name reserved, but should have its ownership transferred to a joint / lead maintainer to manage on crates.io. Not sure what the best setup for that is.

I think there's no pressure to get a first crate version published until we've got some new direction for the API going forward. We can instead "release" as normal pre-releases until then.

Some discussion points from the original issue by @bnewbold:

Under the context that there seem to be very few or no crates depending on this one right now, and it's still at version 0.1.1 > (so breaking updates might be feasible), my priorities would be:

  • update for contemporary Rust (warnings, etc); probably require either 1.26+ or Rust 2018
  • vastly expanded test suite, including test vectors from other implementations
  • review API for best practices (eg, naming)
  • clippy + rustfmt clean
  • informal code quality and security review
  • feature completeness with upstream libmacaroons
  • meeting any requirements and rust best practices to get to a "1.0" API
  • meeting any requirements for inclusion in Debian and other OS-level package managers
  • documentation and examples

Non-priorities (but maybe interesting to others) would be:

  • helpers for verifying caveats (beyond the existing "specific" and "general")
  • no-std support
  • built-in "bakery" API to manage root keys (a la a popular golang library)
  • WASM
  • exposing a C ABI
@bnewbold
Copy link
Member

Thanks for copying these over!

I think expanded test coverage is the highest priority for me, particularly to ensure interoperability with other implementations.

@bbigras
Copy link

bbigras commented May 20, 2021

Could https://crates.io/crates/macaroon point to this repo?

@dunxen
Copy link
Member Author

dunxen commented May 23, 2021

@bbigras We'll just need the ownership of that crate transferred from @jacklund. I'll send an email to him regarding this.

@jacklund
Copy link
Contributor

I added @duncandean as owner. I tried to add @bnewbold, but for some reason it couldn't find their username.

@dunxen
Copy link
Member Author

dunxen commented May 24, 2021

Thanks! I think we'll release a new version that will probably have some breaking changes to the API. We'll need to just discuss what we'd like in that release.

From what's mentioned above, I think a review of the API for best practices and expanding the test suite should be priorities for a 0.2 release. Also a commitment to supporting certain Rust stable minimum version (1.47+ was mentioned in #41 but could be reconsidered now if there are certain features we feel are beneficial).

@bnewbold, I can add you as a crate owner if you provide a username for crates.io :)

@bnewbold
Copy link
Member

Username should match my github username: https://crates.io/users/bnewbold

I just verified that I could still log in to crates.io and don't see any "Owner Invites". You can email me at my github username, domain robocracy.org to debug one-to-one.

@bbigras
Copy link

bbigras commented Sep 16, 2021

Any progress on pointing https://crates.io/crates/macaroon to this repo?

@dunxen
Copy link
Member Author

dunxen commented Sep 17, 2021 via email

@bbigras
Copy link

bbigras commented Sep 21, 2021

Thanks. I'm waiting for this to add macaroon to a project.

@dunxen
Copy link
Member Author

dunxen commented Feb 19, 2022

Meant to post that the crate name now points to this project again.

@bnewbold bnewbold pinned this issue Oct 6, 2022
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

No branches or pull requests

4 participants