[SERVER-29825] Do not allow rename from unreplicated to replicated DB or vice-versa Created: 23/Jun/17 Updated: 30/Oct/23 Resolved: 01/Nov/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication, Storage |
| Affects Version/s: | 3.4.7, 3.5.11 |
| Fix Version/s: | 3.6.10, 4.0.5, 4.1.5 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Matthew Russotto | Assignee: | Vesselina Ratcheva (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||
| Operating System: | ALL | ||||||||||||||||||||
| Backport Requested: |
v4.0, v3.6, v3.4
|
||||||||||||||||||||
| Sprint: | Repl 2018-10-22, Repl 2018-11-05 | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Linked BF Score: | 53 | ||||||||||||||||||||
| Description |
|
We currently allow renameCollection from the "local" database to a replicated database. This doesn't make a lot of sense; the local collections may be different on each replica, so doing so will result in an inconsistent set. We also allow renameCollection from a replicated DB to an unreplicated one; this usually works but will cause problems if the local collection exists and dropExisting isn't specified. |
| Comments |
| Comment by Githook User [ 19/Nov/18 ] |
|
Author: {'name': 'Vesselina Ratcheva', 'email': 'vesselina.ratcheva@10gen.com', 'username': 'vessy-mongodb'}Message: (cherry picked from commit f68d6091bad5a2747cc09c743b8c2c7b4e42a65a) |
| Comment by Githook User [ 13/Nov/18 ] |
|
Author: {'name': 'Vesselina Ratcheva', 'email': 'vesselina.ratcheva@10gen.com', 'username': 'vessy-mongodb'}Message: (cherry picked from commit f68d6091bad5a2747cc09c743b8c2c7b4e42a65a) |
| Comment by Githook User [ 01/Nov/18 ] |
|
Author: {'name': 'Vesselina Ratcheva', 'email': 'vesselina.ratcheva@10gen.com', 'username': 'vessy-mongodb'}Message: |
| Comment by Matthew Russotto [ 23/Jun/17 ] |
|
I think the fuzzer is fine; it caught a real bug. Once it's fixed the fuzzer either has to disallow it or expect it to fail. |
| Comment by Daniel Pasette (Inactive) [ 23/Jun/17 ] |
|
Should this be controlled at the server or should we make a rule to disallow this in the fuzzer |