tonysun83 opened a new pull request #1375: Refactor process_request to not drop 
req
URL: https://github.com/apache/couchdb/pull/1375
 
 
   ## Overview
   Previously, when we fail authorization but pass authentication, we use
   the old HttpReq which drops user_ctx (so it is undefined). This change 
retains the authenticated HttpReq which includes a user_ctx. This allows for 
after_requests to utilize a user_ctx that is not
   undefined.
   
   ## Testing recommendations
   Eunit test needed, but please run chttpd regression bucket. Opening this PR 
to start
   discussion and ensure that this change is warranted.
   
   ## Related Issues or Pull Requests
   https://github.com/apache/couchdb/issues/1374
   
   ## Checklist
   
   - [X] Code is written and works correctly;
   - [ ] Changes are covered by tests;
   - [X] 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

Reply via email to