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

Jump to bottom

Rethink x509.KeyUsage repr() #1834

Closed
alex opened this issue Apr 12, 2015 · 2 comments · Fixed by #1850
Closed

Rethink x509.KeyUsage repr() #1834

alex opened this issue Apr 12, 2015 · 2 comments · Fixed by #1850
Labels
api design good first issue x509
Milestone
Ninth Release

Comments

@alex
Copy link
Member

alex commented Apr 12, 2015

Right now we use N/A for two fields if they're undefined, this is a bit non-conventional, so we should think if there's a better way to express this.
@alex alex added this to the Ninth Release milestone Apr 12, 2015
@reaperhulk
Copy link
Member

What about None?

@alex
Copy link
Member Author

alex commented Apr 15, 2015

That sounds like a solid improvement.

reaperhulk added a commit to reaperhulk/cryptography that referenced this issue Apr 15, 2015
@reaperhulk
improve KeyUsage repr
b372e67 
fixes pyca#1834
@reaperhulk reaperhulk mentioned this issue Apr 15, 2015
Merged
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
api design good first issue x509
Milestone
Ninth Release
Development

Successfully merging a pull request may close this issue.

improve KeyUsage repr reaperhulk/cryptography
2 participants
@alex @reaperhulk