eiri opened a new pull request #645: Pass db open options in fabric_view_map
URL: https://github.com/apache/couchdb/pull/645
 
 
   ## Overview
   
   We don't set db open options in `fabric_view_map` which doesn't matter in 
the most cases, but prevents from opening docs on admin-only databases such as 
`_users`, because of the missing `user_ctx`
   
   This patch passes `user_ctx` set at http request down to a database that 
holds a querying view.  
   
   ## Testing recommendations
   
   We don't have full coverage of this part of fabric, unfortunately. To quick 
test add a simple ddoc to `_users` database, something like:
   
   ```json
   {
     "views": {
       "names": {
         "map": "function(doc) { emit(doc.name); }"
       }
     }
   }
   ```
   
   And then query it with `include_docs=true` that should return all the user 
documents in the users database.
   
   ## Checklist
   
   - [x] Code is written and works correctly;
   - [ ] 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

Reply via email to