Description
SUBJ.
Matt convinced me that indeed we did it wrong:
<alkby> I believe 1.7 and 1.6 always demanded admin
<alkby> and 1.8 too
<ingenthr> that's a real problem from a client perspective, since someone using a client is just using a bucket
<alkby> right but flush is a very destructive operation
<ingenthr> if buckets are in fact about multitenancy, then the tenant should be able to throw away their contents
<ingenthr> yep, I know.
<alkby> for unit tests you set it up for bucket with admin credentials
<alkby> well, tenant is good argument
<ingenthr> this means it's not functionally equivalent to memcached flush though, and that was the whole point of MB-5170
<ingenthr> that we couldn't do memcached flush safely, so we'd replace it with RESTful flush
<alkby> you're right
<alkby> lets file a bug and address it asap. Thanks for raising this
<ingenthr> but if restful flush is different semantically (you can flush this only with super creds)
<ingenthr> okay, will do, thanks
<alkby> I'll file bug
Attachments
Issue Links
- blocks
-
JCBC-173 flush will not work owing to MB-7381
- Resolved
For Gerrit Dashboard: MB-7381 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
22445,7 | JCBC-144: Implement flush over HTTP | master | couchbase-java-client | Status: MERGED | +2 | +1 |
23592,2 | MB-7381: allow flush with bucket credentials | master | ns_server | Status: MERGED | +2 | +1 |
23930,2 | [bp] MB-7381: allow flush with bucket credentials | 2.0.1 | ns_server | Status: MERGED | +2 | +1 |
23967,1 | Merge remote-tracking branch 'origin/2.0.1' | master | ns_server | Status: MERGED | +2 | +1 |
24557,1 | Merge remote-tracking branch 'couchbase/master' into b202 | 2.0.2 | ns_server | Status: MERGED | +2 | +1 |