Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Handle Unicode domains in Plan and DomainFilter #2

Closed
wants to merge 2 commits into from

Conversation

kimsondrup
Copy link

Description

Ensure support for Internationalized Domain Names for Applications (aka. domains using Unicode) using golang.org/x/net/idna

Disclaimer, this is my first Go code so please look at it with extra skepticism

Checklist

  • Unit tests updated
  • [] End user documentation updated

@kimsondrup kimsondrup closed this Aug 19, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 10457917979

Details

  • 10 of 12 (83.33%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 52.583%

Changes Missing Coverage Covered Lines Changed/Added Lines %
plan/plan.go 2 4 50.0%
Totals Coverage Status
Change from base Build 10419810182: 0.005%
Covered Lines: 14749
Relevant Lines: 28049

💛 - Coveralls

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants