nickva commented on code in PR #6190:
URL: https://github.com/apache/couchdb/pull/6190#discussion_r4449716785
##########
src/chttpd/src/chttpd.erl:
##########
@@ -426,7 +426,7 @@ handle_req_after_auth(HandlerKey, HttpReq) ->
fun chttpd_db:handle_request/1
),
AuthorizedReq = chttpd_auth:authorize(
- possibly_hack(HttpReq),
Review Comment:
I added a separate commit to the PR with some of the tests cases and with an
attempt to simplify the error cases to just one "invalid endpoint" case. Feel
free to remove the commit or pick and chose if anything looks good there or
combine with yours.
Another thing we should add is a doc update indicated what changed.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
