Skip to content

NETWORKING:Def CName in Http Publish Addr to True#33631

Merged
original-brownbear merged 1 commit intoelastic:masterfrom
original-brownbear:32806-follow-up
Sep 18, 2018
Merged

NETWORKING:Def CName in Http Publish Addr to True#33631
original-brownbear merged 1 commit intoelastic:masterfrom
original-brownbear:32806-follow-up

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

* Follow up to #32806 setting the setting to true for 7.x
@original-brownbear original-brownbear added >non-issue :Distributed/Network Http and internode communication implementations v7.0.0 labels Sep 12, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra

@original-brownbear
Copy link
Copy Markdown
Contributor Author

This PR changes the behavior to what was requested here #32806 (review) for 7.

Copy link
Copy Markdown
Contributor

@Tim-Brooks Tim-Brooks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@original-brownbear
Copy link
Copy Markdown
Contributor Author

@tbrooks8 thanks!

@original-brownbear original-brownbear merged commit 87cedef into elastic:master Sep 18, 2018
@original-brownbear original-brownbear deleted the 32806-follow-up branch September 18, 2018 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Network Http and internode communication implementations >non-issue v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants