[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:
Backports
Related
related to SERVER-32205 Dropping admin db sets FCV to 3.4 but... Closed
is related to SERVER-31793 Add a hook to check that all collecti... Closed
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: SERVER-32126 check UUID in validate

(cherry picked from commit 003dfbdcdd2ce5e33cd92ef57701e98766d8d192)
Branch: v3.6
https://github.com/mongodb/mongo/commit/9f490e3d574090f8bac9b92117c026162cd058a0

Comment by Maria van Keulen [ 06/Feb/18 ]

I am requesting a backport for this change because as of SERVER-32741, the featureCompatibilityVersion parameter defaults to 3.6 so the master branch is no longer able to catch 3.4-3.6 conversion bugs.

Comment by Githook User [ 18/Dec/17 ]

Author:

{'name': 'Kevin Albertson', 'email': 'kevin.albertson@10gen.com', 'username': 'kevinAlbs'}

Message: SERVER-32126 check UUID in validate
Branch: master
https://github.com/mongodb/mongo/commit/003dfbdcdd2ce5e33cd92ef57701e98766d8d192

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 SERVER-31793?

Generated at Thu Feb 08 04:29:16 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.