-
Notifications
You must be signed in to change notification settings - Fork 41
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
sigstore: Create API stubs #8
Conversation
sigstore/rekor/_client.py
Outdated
""" | ||
|
||
|
||
class RekorClient: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rekor's API seems to be a lot more complicated. The Rekor client in Go is generated via Swagger here.
We might have to figure out the main things we want/need to support for the beta.
sigstore/fulcio/_client.py
Outdated
Client implementation for interacting with Fulcio. | ||
""" | ||
|
||
from dataclasses import dataclass |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These types are modelled after:
https://github.com/sigstore/sigstore-java
https://github.com/sigstore/fulcio/blob/main/pkg/api/client.go
@di @woodruffw
I'm thinking of adding |
I think the importable, public API should probably be |
Sure, but what about the tool itself? It's ok to use it like |
…ck (sigstore#8) * git, dependabot, conformance: add sigstore-python nightly submod, track Signed-off-by: William Woodruff <william@trailofbits.com> * dependabot: remove custom Docker registry Seems to only be needed when the registry requires a password, which this one doesn't. Signed-off-by: William Woodruff <william@trailofbits.com> Signed-off-by: William Woodruff <william@trailofbits.com>
Closes #7