[SERVER-73446] DatabaseNameUtil::parseTenantIdFromDatabaseName() should be moved to tenant migration util file. Created: 30/Jan/23 Updated: 30/Jan/23 Resolved: 30/Jan/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Suganthi Mani | Assignee: | [DO NOT USE] Backlog - Server Serverless (Inactive) |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Serverless
|
||||||||||||
| Participants: | |||||||||||||
| Description |
|
DatabaseNameUtil::parseTenantIdFromDatabaseName() tries to parse tenantID when gMultitenancySupport is disabled which is not very safe. We introduced that function for tenant migration as we night need to parse the tenantID even when gMultitenancySupport is disabled. In future, if parseTenantIdFromDatabaseName() is called outside of tenant migration code, we can cause bugs due to parsing tenantId w/ gMultitenancySupport disabled. To prevent this, we should move that function to tenant migration util file. |