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

wireaddr: allow for UpperCase DNS names #5792

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

m-schmoock
Copy link
Collaborator

This further relaxes the DNS hostname checks to allow for uppercase input.

Changelog-None

This further relaxes the DNS hostname checks to allow for uppercase
input.

Changelog-None
@m-schmoock m-schmoock requested a review from cdecker December 7, 2022 10:16
@m-schmoock m-schmoock added this to the v22.11.1 milestone Dec 7, 2022
Copy link
Contributor

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack 8cc6ce5

@cdecker cdecker enabled auto-merge (rebase) December 7, 2022 11:42
@cdecker cdecker merged commit 0ae6f4d into ElementsProject:master Dec 7, 2022
@m-schmoock m-schmoock deleted the wireaddr/uppercase_dns branch December 8, 2022 09:38
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.

3 participants