Skip to content

Conversation

@VijayanB
Copy link
Member

@VijayanB VijayanB commented Jul 8, 2021

Signed-off-by: Vijayan Balasubramanian [email protected]

Description

Since OpenSearch supports TLS certificate authentication,
this plugin should support those mechanism as well.

Issues Resolved

#30

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has documentation added
  • Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Since OpenSearch supports TLS certificate authentication,
this plugin should support those mechanism as well.

Signed-off-by: Vijayan Balasubramanian <[email protected]>
@VijayanB VijayanB added the enhancement Increases software capabilities beyond original client specifications label Jul 8, 2021
@VijayanB VijayanB self-assigned this Jul 8, 2021
@VijayanB VijayanB requested review from jmazanec15 and vamshin July 8, 2021 23:09
Copy link
Member

@jmazanec15 jmazanec15 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@vamshin vamshin left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

@VijayanB VijayanB merged commit fa86342 into opensearch-project:main Jul 12, 2021
This was referenced Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Increases software capabilities beyond original client specifications

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants