Skip to content

Update configuration.asciidoc #26534

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

Closed
wants to merge 1 commit into from
Closed

Conversation

john-mcpeek
Copy link

@john-mcpeek john-mcpeek commented Sep 7, 2017

This example doesn't explain where the sslcontext comes from and doesn't use the keystore (truststore) that you loaded just above it. This was very confusing until I figured that out.

This example doesn't explain where the sslcontext comes from and doesn't use the keystore (truststore) that you loaded just above it. This was very confusing until I figured that out.
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@jasontedor jasontedor requested a review from javanna September 7, 2017 14:28
@javanna javanna added >docs General docs changes :Clients/Java Low Level REST Client Minimal dependencies Java Client for Elasticsearch review labels Sep 18, 2017
@javanna javanna self-assigned this Sep 18, 2017
@javanna javanna added the v5.5.4 label Sep 18, 2017
@javanna
Copy link
Member

javanna commented Sep 18, 2017

hi @john-mcpeek have you signed our CLA?

javanna added a commit to javanna/elasticsearch that referenced this pull request Sep 19, 2017
… client docs

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to elastic#26534
Closes elastic#26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs (#26705)

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs (#26705)

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs (#26705)

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs (#26705)

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
javanna added a commit that referenced this pull request Sep 25, 2017
… client docs

Include SSLContext creation and add the missing "https" scheme to the host creation.

Relates to #26534
Closes #26670
@javanna
Copy link
Member

javanna commented Sep 25, 2017

This was fixed with #26705

@javanna javanna closed this Sep 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting CLA :Clients/Java Low Level REST Client Minimal dependencies Java Client for Elasticsearch >docs General docs changes v5.5.4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants