Skip to content

Add support for RFC3779 X509 extensions for IPs and AS-IDs#245

Open
niklbird wants to merge 4 commits intowbond:masterfrom
niklbird:master
Open

Add support for RFC3779 X509 extensions for IPs and AS-IDs#245
niklbird wants to merge 4 commits intowbond:masterfrom
niklbird:master

Conversation

@niklbird
Copy link
Copy Markdown

No description provided.

@wbond
Copy link
Copy Markdown
Owner

wbond commented Nov 11, 2022

Looks like you need to fix some linting errors

@niklbird
Copy link
Copy Markdown
Author

Looks like you need to fix some linting errors

Thank you! I fixed them now :)

@niklbird
Copy link
Copy Markdown
Author

niklbird commented Dec 6, 2022

Hey :) Is there anything still missing that you need?

@wbond
Copy link
Copy Markdown
Owner

wbond commented Aug 23, 2023

Could you rebase this on master?

@wbond
Copy link
Copy Markdown
Owner

wbond commented Aug 23, 2023

Thanks for taking the time to add these.

Overall it looks good. The one style change I would request is that for this project we use under_score notation instead of camelCase for the ASN.1 field names. In other words, things like addressRange should be address_range.

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.

2 participants