Skip to content

Test if the user id migration script breaks anything in our stack #1589

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
4 tasks
pahatz opened this issue Apr 9, 2025 · 1 comment
Open
4 tasks

Test if the user id migration script breaks anything in our stack #1589

pahatz opened this issue Apr 9, 2025 · 1 comment
Assignees

Comments

@pahatz
Copy link
Contributor

pahatz commented Apr 9, 2025

From the migration script for removing user ids issue #1123
Run the query and check if anything breaks in the services. This should be done in the branch remove-user-ids-from-filepaths
things to test: run the integration test suite.

  • create issues for things that are broken on main
    • apps need to prepend the username to the file path when interacting with the inbox - fixed in feature/do-not-leak-user-ids
    • inbox needs to remove the username from the file path in the message - fixed in feature/do-not-leak-user-ids
  • demo broken things
@jbygdell jbygdell self-assigned this Apr 10, 2025
@jbygdell
Copy link
Collaborator

UPDATE sda.files SET submission_file_path = REPLACE(submission_file_path, CONCAT(REPLACE(submission_user, '@', '_'),'/'), '');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants