Skip to content
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

Unknown hostnames correspond to reserved ones in RFC 2602 #2087

Merged
merged 2 commits into from Jun 19, 2023

Conversation

vitalijr2
Copy link
Collaborator

Fixes #2081

The Ukrainian mobile operator Lifecell resolves non-existent hostnames and returns fake 404 pages for some reason. There is a possibility that some other providers can do such strange things as well.

RFC 2602 reserves some top level domain names for test purpose. To avoid such issues like #2081 I propose to replace all non-existent domain names in LoggerTest.

@vitalijr2 vitalijr2 requested review from velo and kdavisk6 June 16, 2023 05:53
@velo velo merged commit ad167f5 into OpenFeign:master Jun 19, 2023
3 checks passed
@vitalijr2 vitalijr2 deleted the logger-test-with-error branch June 23, 2023 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LoggerTest is failed: is it just me or do you have the same errors?
2 participants