[SERVER-32126] validate() should do basic sanity checks for UUIDs Created: 30/Nov/17 Updated: 30/Oct/23 Resolved: 18/Dec/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Code |
| Affects Version/s: | 3.6.0-rc7 |
| Fix Version/s: | 3.6.4, 3.7.1 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Robert Guo (Inactive) | Assignee: | Kevin Albertson |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Backwards Compatibility: | Minor Change | ||||||||||||||||
| Backport Requested: |
v3.6
|
||||||||||||||||
| Sprint: | TIG 2018-1-1, TIG 2017-12-18 | ||||||||||||||||
| Participants: | |||||||||||||||||
| Description |
|
validate() should do some basic sanity checks for UUIDs. E.g. check UUIDs exist (under FCV 3.6+) and that the UUID format is valid and not corrupted. |
| Comments |
| Comment by Githook User [ 26/Feb/18 ] |
|
Author: {'email': 'kevin.albertson@10gen.com', 'name': 'Kevin Albertson', 'username': 'kevinAlbs'}Message: (cherry picked from commit 003dfbdcdd2ce5e33cd92ef57701e98766d8d192) |
| Comment by Maria van Keulen [ 06/Feb/18 ] |
|
I am requesting a backport for this change because as of |
| Comment by Githook User [ 18/Dec/17 ] |
|
Author: {'name': 'Kevin Albertson', 'email': 'kevin.albertson@10gen.com', 'username': 'kevinAlbs'}Message: |
| Comment by Eric Milkie [ 30/Nov/17 ] |
|
After reviewing the comments in that ticket, it seems to me that it simply didn't occur to anyone, not that there was an explicit reason why adding it to validate would be inadvisable. |
| Comment by Max Hirschhorn [ 30/Nov/17 ] |
|
milkie, geert.bosch, maria.vankeulen, is there a reason we didn't consider changing the implementation of the "validate" command for how to go about doing |