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

Use the INTL_IDNA_VARIANT_UTS46 idn_to_ascii variant #192

Closed
wants to merge 1 commit into from
Closed

Use the INTL_IDNA_VARIANT_UTS46 idn_to_ascii variant #192

wants to merge 1 commit into from

Conversation

mtorromeo
Copy link
Contributor

The default variant INTL_IDNA_VARIANT_2003 is deprecated in php 7.2.
The default will be changed to INTL_IDNA_VARIANT_UTS46 in php 7.4 so we should use that if it is defined.
This is similar to how it was changed in Zend\Validator\EmailAddress.

The default variant INTL_IDNA_VARIANT_2003 is deprecated in php 7.2.
The default will be changed to INTL_IDNA_VARIANT_UTS46 in php 7.4 so we should use that if it is defined.
This is similar to how it was changed in `Zend\Validator\EmailAddress`.
@Slamdunk
Copy link
Contributor

Duplicate of #181 and #189

@Slamdunk Slamdunk mentioned this pull request Mar 1, 2018
3 tasks
@Ocramius Ocramius self-assigned this Mar 1, 2018
@Ocramius Ocramius added the bug label Mar 1, 2018
@Ocramius Ocramius added this to the 2.9.0 milestone Mar 1, 2018
@Ocramius
Copy link
Member

Ocramius commented Mar 1, 2018

Handled in #189

@Ocramius Ocramius closed this Mar 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants