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

[WFLY-17214] Prefer hostname than IP in http-connector host #16228

Closed
wants to merge 1 commit into from

Conversation

gaol
Copy link
Contributor

@gaol gaol commented Oct 31, 2022

Thanks for submitting your Pull Request!

Please delete this text, and add a link to the Jira issue solved by this PR.

If this PR is not for the 'main' branch you must add a link to the equivalent change in 'main'.

Remember to use the Jira issue ID in the PR title and any commits.

Issue: https://issues.redhat.com/browse/WFLY-17214

@wildfly-ci
Copy link

Hello, gaol. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

@gaol
Copy link
Contributor Author

gaol commented Oct 31, 2022

@ehsavoie would you please review ? thanks

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Oct 31, 2022
@bstansberry
Copy link
Contributor

/ok-to-test

@gaol gaol requested a review from ehsavoie November 7, 2022 03:56
@gaol
Copy link
Contributor Author

gaol commented Nov 7, 2022

I am closing it after discussion with @ehsavoie , it would bring behavior change in case of remote JMS client.

@gaol gaol closed this Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
3 participants