Jason Parraga
Just tested this locally and it works fine for basic auth workflows.
> Thanks @Sovietaced for your contribution, is this a similar problem https://github.com/opensearch-project/terraform-provider-opensearch/issues/102 that can be solved with this PR? > > @bbarani Possibly. I'm not sure if #102 has been...
> I see one of your another PR #114 (~ same subject), any co-relation between these 2 PR's? They are related. I made them separate pull requests because I wasn't...
> Thanks @Sovietaced, Can you please add some unit tests to test this change? Thanks I had looked into adding unit tests but I don't see any existing coverage for...
> Looks to me like if this PR goes in we dont have to explicitly disable the request signing right ? This PR #114 mentions to disable it. > @Sovietaced...
I will just prepare this pull request to remove the option to disable request signing.
> Thanks @Sovietaced I will go ahead and merge the documentation PR #114, please try to add some unit tests for this PR and then we can proceed to merge...
@prudhvigodithi I took a look at the test code here and I don't think there is much I can really do. There exists zero unit test code that targets basic...
It appears this happens when health checking is disabled. If health checking is enabled, a 401 is returned.
> [Untriage] > Thanks @Sovietaced, there are some PR's went in with fix related to basic auth #87 #76, I dont think they fix this `invalid character` error, can you...