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
Test for PR #9486, Validate allowed TLDs with current IANA list #9674
Conversation
c0307ed
to
b2f16c1
Compare
@@ -8,6 +8,9 @@ | |||
|
|||
namespace Piwik\Tests\Integration; | |||
|
|||
use Guzzle\Http\Client; | |||
use Guzzle\Http\Message\Request; | |||
use Guzzle\Http\Message\Response; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI: While guzzle is a library in Piwik it is only there for AWS-SDK. It's not a supported lib and we might remove it at some point. Instead use Piwik\Http
.
I will fix those soon. |
b2f16c1
to
88ad8c9
Compare
Introduced CR suggestions. |
@sebastianpiskorski @tsteur This PR introduces a little challenge, see: Automated test for allowed TLDs starts failing when iana.org starts supporting new domain extensions #9977 |
I think it's supposed to fail in such a case to make sure it'll work for all domains in the future. That's why I merged it. Not sure how often they add new domains though. If they add new ones too often could maybe solve it differently |
fixes #9486