Skip to content
This repository has been archived by the owner on Jan 31, 2020. It is now read-only.

Changes done for adding support for "ø" for DK idn #150

Merged
merged 3 commits into from
Mar 17, 2017

Conversation

damanikrish
Copy link

When ever I try to create Uri object for url string "http://www.fitnessø.dk/", it throws an exception with message "Host "www.fitnessø.dk" is not valid or is not accepted by...". And when debugged the source, I came to know that this error is because it is not validated by HostName validator in Uri::isValidDnsHostname().

@akrabat
Copy link
Contributor

akrabat commented Feb 24, 2017

Travis failure is due to zendframework/zend-servicemanager#182, not this PR.

@akrabat akrabat self-requested a review February 24, 2017 09:36
@akrabat
Copy link
Contributor

akrabat commented Feb 24, 2017

Are there any other characters that should be allowed in a DK url?

@damanikrish
Copy link
Author

@akrabat , I am not sure for the other characters, I just came across the use case for this character and hence created this PR for the same.

Thanks,
Krish Damani

@damanikrish
Copy link
Author

@akrabat , I have added two more characters and updated the PR,
Reference for allowed characters in dk domain names https://www.dk-hostmaster.dk/sites/default/files/Characters%20allowed%20in.pdf

Thanks,
Krish Damani

@Xerkus Xerkus force-pushed the bugfix/support-for-ø-in-dk branch from 5520a29 to 0c4c3df Compare March 17, 2017 09:47
@Xerkus Xerkus merged commit 0c4c3df into zendframework:master Mar 17, 2017
Xerkus added a commit that referenced this pull request Mar 17, 2017
Xerkus added a commit that referenced this pull request Mar 17, 2017
@Xerkus Xerkus added this to the 2.9.0 milestone Mar 17, 2017
@Xerkus Xerkus added the bug label Mar 17, 2017
Xerkus added a commit that referenced this pull request Mar 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants