eiri opened a new pull request #1588: Check if db exists in /db/_ensure_full_commit call URL: https://github.com/apache/couchdb/pull/1588 ## Overview We removed a security call in `do_db_req` in apache/couchdb#1246 to avoid duplicate authorization check and as a result there are now no db validation in noop call `/db/_ensure_full_commit`. This makes it always return a success code, even for the missing databases. This fix places the security check directly in `_ensure_full_commit` call and adds eunit tests for a good measure. ## Testing recommendations `make eunit apps=chttpd suites=chttpd_db_test` for isolated tests ## Related Issues or Pull Requests Closes apache/couchdb#1585 ## Checklist - [x] Code is written and works correctly; - [x] Changes are covered by tests; - [ ] Documentation reflects the changes;
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
