Actions
Bug #24220
closed
User session database information about tenants should not break in 8.1
Pull Request:
Severity:
Critical - prevents main use of Rudder | no workaround | data loss | security
UX impact:
User visibility:
Effort required:
Small
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
No
Description
We added a tenants column in the usersessions table in #23440 because we plan to store tenants information each time a user logs in.
But in the 7.3 version of the table introduced in #24146, the column does not yet exist.
- we need a migration for the
tenantscolumn with empty tenants by default. - we also need to change the type of stored tenants to a list of tenant ids, introduced in #24140
Updated by Clark ANDRIANASOLO 10 months ago
- Related to User story #23440: Add users table to better track user and sessions added
- Related to Architecture #24146: Backport user table to Rudder 7.3 added
- Related to User story #24140: Have a source of known-tenants in rudder-core added
Updated by Clark ANDRIANASOLO 10 months ago
- Status changed from New to In progress
Updated by Clark ANDRIANASOLO 10 months ago
Work in progess here: https://github.com/clarktsiory/rudder/commit/8ce9dc8abe9e2fcba275d08ad5ac7f2957a38f72
Updated by Clark ANDRIANASOLO 4 months ago
- Status changed from In progress to Rejected
This is no longer an issue, as migration from 7.3 to 8.1 was seamless, and no longer causes database query issues : queried columns were made explicit in #24344.
Having a list of TenantId is not required and we do not need to bother for now
Actions