Skip to content

Conversation

@sftim
Copy link

@sftim sftim commented Aug 9, 2021

Allow requests against any Elasticsearch-like API.

This reverts pull request #1623

@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?

@cla-checker-service
Copy link

cla-checker-service bot commented Aug 9, 2021

💚 CLA has been signed

@sftim
Copy link
Author

sftim commented Aug 9, 2021

Also relevant to #1648

@sftim
Copy link
Author

sftim commented Aug 9, 2021

Getting this one in before @elasticmachine learns how to auto-reject them

@sftim sftim force-pushed the remove_elasticsearch_verification branch 2 times, most recently from cd41efc to 7564c34 Compare August 9, 2021 17:29
@sftim sftim force-pushed the remove_elasticsearch_verification branch from 7564c34 to c238b8f Compare August 9, 2021 17:30
@sethmlarson sethmlarson closed this Aug 9, 2021
@orenmazor
Copy link

booourns. I believed in you, @sftim

@JPrevost
Copy link

JPrevost commented Aug 9, 2021

Oh look, we got a no comment close. Glad they are willing to discuss this... who's gonna fork?

@sftim
Copy link
Author

sftim commented Aug 16, 2021

#1676 (comment) suggests an alternative patch with similar aims

@orenmazor
Copy link

#1676 (comment) suggests an alternative patch with similar aims

yeah it's inevitable. this is damage. we'll route around this one way or another 🤷

@sftim
Copy link
Author

sftim commented Aug 16, 2021

#1639 (comment) explains(?) some of the background for the behavior that this PR reverts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants