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

Fix: Email Validator vs IDN #7281

Merged
merged 1 commit into from
Mar 10, 2015
Merged

Fix: Email Validator vs IDN #7281

merged 1 commit into from
Mar 10, 2015

Conversation

svyatoslav-kubakh
Copy link

Non-latin emails validation

@svyatoslav-kubakh
Copy link
Author

I can not find the cause of the Travis CI fails, but locally everything is OK:

ubuntu@php56:/var/www/ksar-zf2/tests$ php -v
PHP 5.6.6-1+deb.sury.org~utopic+1 (cli) (built: Feb 20 2015 11:25:37) 
Copyright (c) 1997-2015 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2015 Zend Technologies
    with Zend OPcache v7.0.4-dev, Copyright (c) 1999-2015, by Zend Technologies
ubuntu@php56:/var/www/ksar-zf2/tests$ cat TestConfiguration.php | grep TESTS_ZEND_VALIDATOR_ONLINE_ENABLED
defined('TESTS_ZEND_VALIDATOR_ONLINE_ENABLED') || define('TESTS_ZEND_VALIDATOR_ONLINE_ENABLED', true);
ubuntu@php56:/var/www/ksar-zf2/tests$ php -m | grep intl
intl
ubuntu@php56:/var/www/ksar-zf2/tests$ phpunit.phar ZendTest/Validator/EmailAddressTest.php 
PHPUnit 4.5.0 by Sebastian Bergmann and contributors.
Configuration read from /var/www/ksar-zf2/tests/phpunit.xml.dist
........................................
Time: 6.04 seconds, Memory: 5.00Mb
OK (40 tests, 233 assertions)

@weierophinney weierophinney merged commit fb5590c into zendframework:develop Mar 10, 2015
weierophinney added a commit that referenced this pull request Mar 10, 2015
weierophinney added a commit that referenced this pull request Mar 10, 2015
@weierophinney weierophinney added this to the 2.4.0 milestone Mar 10, 2015
weierophinney added a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
…ubakh/hotfix/email-validator-idn

Fix: Email Validator vs IDN
weierophinney added a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
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.

2 participants