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

remove default port 9200; #822

Closed
wants to merge 1 commit into from
Closed

Conversation

shaowangzhen
Copy link

don't want to set port when use domain;

don't want to set port when use domain;
@polyfractal
Copy link
Contributor

See comment in: #309 (comment)

Basically, the client is supposed to remain un-opinionated and reflect the defaults of Elasticsearch. ES uses port 9200 (not port 80), so it should default to that. If you want to use port 80, you'll need to set it explicitly, or write a wrapper around the client to do that by default for you.

@polyfractal polyfractal closed this Jan 7, 2019
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.

2 participants