search

opensearch-project / terraform-provider-opensearch

https://registry.terraform.io/providers/opensearch-project/opensearch
Apache License 2.0
73 stars 56 forks source link

Improve documentation for HTTP basic authentication #114

Closed Sovietaced closed 9 months ago

Sovietaced commented 10 months ago

Description

This pull request updates the documentation with examples for configuring the provider to use HTTP basic authentication. I classified this under the "static credentials" section but I'm happy to reorganize things. The example is particularly useful because request signing is enabled by default and will result in mysterious HTTP 403 errors, ie 

│ Error: HTTP 403 Forbidden: Permission denied. Please ensure that the correct credentials are being used to access the cluster.

Issues Resolved

107

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.

Sovietaced commented 10 months ago

I am wondering if I should file a separate issue so that the provider is intelligent enough to disable request signing when it detects that HTTP basic auth credentials are configured.

prudhvigodithi commented 9 months ago

Please take a look at my comment here https://github.com/opensearch-project/terraform-provider-opensearch/pull/117#issuecomment-1789702879.

prudhvigodithi commented 9 months ago

LGTM based on this comment https://github.com/opensearch-project/terraform-provider-opensearch/pull/117#issuecomment-1791415942, merging this PR, thanks @Sovietaced.