[GitHub] [incubator-druid] justinborromeo opened a new issue #7369: Change SQL endpoint response code from 405 to 404 on non-broker processes

[GitBox <gi...@apache.org>]

justinborromeo opened a new issue #7369: Change SQL endpoint response code from 405 to 404 on non-broker processes
URL: https://github.com/apache/incubator-druid/issues/7369
 
 
   ### Description
   
   Currently, when the `/druid/v2/sql` endpoint is hit on a non-broker process, a 405 Method Not Found status code is returned.  I feel as if a 405 is misleading since it implies that another method will work.  Perhaps a better option would be a 404 Resource Not Found as it shows that the resource doesn't exist on that process.
   
   ### Motivation
   
   This change is motivated by [this question](https://groups.google.com/forum/#!topic/druid-user/Trlp6I-1m8w) on the Druid user group.  For beginners especially, the 405 is confusing if they copy the exact cURL command from the docs and send it to the wrong process.
   
   Changing the response code would likely cause compatibility issues with existing clients.  Due to this messiness, I'm opening this issue as a discussion rather than a proposal.  Do people think this change would have value and would have enough value to justify the compatibility headache?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org