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

Add TLSA support #92

Merged
merged 2 commits into from Apr 18, 2023
Merged

Add TLSA support #92

merged 2 commits into from Apr 18, 2023

Conversation

buffrr
Copy link
Contributor

@buffrr buffrr commented Apr 9, 2023

Closes #91

@buffrr
Copy link
Contributor Author

buffrr commented Apr 18, 2023

@silverwind Could you take a look I need to use this soon. Thanks!

@silverwind
Copy link
Collaborator

I would have encoded certificate data as string, but it seems the module uses buffer for other similar fields, so I guess it's at least consistent.

@silverwind silverwind merged commit f14f483 into mafintosh:master Apr 18, 2023
9 checks passed
@buffrr buffrr deleted the tlsa branch April 19, 2023 02:09
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.

add support for TLSA record
2 participants