Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support Cross-Origin resource in http/rest module #218

Closed
lukas-vlcek opened this issue Jun 9, 2010 · 1 comment
Closed

Support Cross-Origin resource in http/rest module #218

lukas-vlcek opened this issue Jun 9, 2010 · 1 comment
Labels
>feature v0.09.0

Comments

@lukas-vlcek
Copy link
Contributor

There is a known issue with Firefox sending extra OPTIONS request first when XHR POST request is created and executed (http://stackoverflow.com/questions/1099787/jquery-ajax-post-sending-options-as-request-method-in-firefox/2865924#2865924). This is causing problems with all POST requests in ES REST API sent from FF.

Moreover, current implementation in ES is causing OPTIONS requests to fall back to GET requests (see NettyHttpRequest.method()) which results in "No handler found for uri [request.uri()] and method [GET]".

As far as I understand it is important to respond with "Access-Control-Allow-Origin: *" to OPTIONS requests.
@kimchy
Copy link
Member

kimchy commented Jul 19, 2010

Support Cross-Origin resource in http/rest module, closed by 5f0470d.

@rgarcia6520 rgarcia6520 mentioned this issue Aug 16, 2019
Closed
ClaudioMFreitas pushed a commit to ClaudioMFreitas/elasticsearch-1 that referenced this issue Nov 12, 2019
Merge pull request elastic#218 from gingerwizard/master
e8431e0 
Allows unauthenticated repos
williamrandolph pushed a commit to williamrandolph/elasticsearch that referenced this issue Jun 4, 2020
@costin
Fix invalid request when using both dynamic and custom headers
dbcc546 
close elastic#218

(cherry picked from commit 81dd5db22dc24a18fc0c23bba8b64f34b170d914)
mindw pushed a commit to mindw/elasticsearch that referenced this issue Sep 5, 2022
Merged in dev/can/ems-timeout-2021-08-31 (pull request elastic#218)
e28b34b 
Parameterize EMS healthcheck timeout

Allows devs to set these higher in tfvars for development

Approved-by: Andre Sodermans
costin pushed a commit that referenced this issue Dec 6, 2022
@elasticsearchmachine
Merge pull request #218 from elastic/main
bca0c29 
🤖 ESQL: Merge upstream
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>feature v0.09.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kimchy @lukas-vlcek