davisp commented on issue #2666: soft-deletion for database
URL: https://github.com/apache/couchdb/pull/2666#issuecomment-607954397
 
 
   Also, a couple quick thoughts on the HTTP API. Looking at it a bit harder I 
realized that we've basically duplicated some functionality because these two 
requests should be functionally equivalent:
   
   ```
   GET /_deleted_dbs/DbName`
   GET /_deleted_dbs?key=DbName`
   ```
   
   And I could see a convincing argument that the second form would be more 
"natural" within the existing APIs for views and so on.
   
   I've also heard some concerns that having the `undelete` vs `delete` 
semantics of our `POST /_deleted_dbs` setup makes auditing traffic 
hard/impossible without introspecting the request body which seems like a 
legitimate critique. How about if we tweaked things to be like such:
   
   * `GET /_deleted_dbs` - List deleted dbs infos as currently.
   * `POST /_deleted_dbs` - Only accepts the `undelete` JSON body
   * `DELETE /_deleted_dbs/DbName?timestamp=2020-01-01T00:00:00` - Remove 
deleted db
   
   /cc @mikerhodes @ricellis @nickva

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to