[SERVER-84004] Add FCV gating to dots and dollars validation on 5.0 Created: 08/Dec/23 Updated: 02/Feb/24 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | 5.0.23 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Alya Berciu | Assignee: | Backlog - Query Optimization |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Assigned Teams: |
Query Optimization
|
||||
| Sprint: | QO 2024-02-05, QO 2024-02-19 | ||||
| Participants: | |||||
| Description |
|
See HELP-53040 for motivation, as well as the docs. There was an oversight in PM-1856 where the logic to decide whether or not to validate . and $ fields is only dependent on the feature flag, but not the FCV version (see the scope of PM-1856). The fix for this should involve updating all checks to use `isEnabled()` instead of `isEnabledAndIgnoreFCV()`, and adding a test to verify that the FCV behavior is correct when upgrading/downgrading. |