-
Notifications
You must be signed in to change notification settings - Fork 91
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.
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
Gateway 504 issue #220
Comments
That's not quite clear, imho. The HTTP 504 error message is being generated by Apache due to a CGI timeout. Hence the Here's the response as seen in wireshark:
There are two topics here which need to be addressed:
Regarding the error message: CORS headers could be set via .htaccess (see http://enable-cors.org/server_apache.html). However, this competes with the headers we've already added in web_output.cc. Firefox as an example returns an error message, if Access-Control-Allow-Origin occurs twice in the HTTP header (even when it's the same value). On the other hand, some of the error messages are solely created on by apache, so it could make sense to move the whole Access-Control-Allow-Origin: *, etc. part to the apache config instead. (just an example for .htaccess:)
See also http://awesometoast.com/cors/ Question: can we provide some kind of (compile time?) flag for web_output.cc to not send any CORS header? Due to a number of dependencies, this topic probably has to wait until the out_csv_finetuning branch is done and eventually merged into master. |
How 504 should be interpreted as user? "internal bug, retry immediately"? "you are overusing service, sleep for several minutes and retry"? |
Apache has hit its timeout value, b/c Overpass didn't respond in time. So either there's too much load on the server, or your query is simply too large and takes too much time. I'd recommend to double check your query first. Maybe run it on a smaller area, split it up into smaller chunks, something along those lines (depends on your query, obviously). |
Follow up issue for tyrasd/overpass-turbo#171 (comment)
The text was updated successfully, but these errors were encountered: