Skip to content

Implement user auto-deletion follow up PR: Update TankConfig to account for XMLConfig changes#417

Merged
Zakaria-Kofiro merged 19 commits intomasterfrom
zkofiro/SRE-35508
Aug 19, 2025
Merged

Implement user auto-deletion follow up PR: Update TankConfig to account for XMLConfig changes#417
Zakaria-Kofiro merged 19 commits intomasterfrom
zkofiro/SRE-35508

Conversation

@Zakaria-Kofiro
Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro commented Aug 19, 2025 •
edited
Loading

Implement user auto-deletion follow up PR: Update TankConfig to account for XMLConfig changes

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

zkofiro and others added 19 commits July 11, 2025 19:11
feat(api): Add /user endpoint to Tank API
0d27cb8
core access/delete logic simplified
5711a75
successfully anonymize + reset user roles for deleted_user
68a9921
fix(api): small updates to controller code
843dd44
fix(api): correct token auth logic
f43e4f2
feat(api): added admintokenservice + models for users
0fc87b6
updates to user API implementation to accept service requests
6ec4ab5
add impl + model code
8a35681
@kevin-mcgoldrick
Merge branch 'master' into zkofiro/SRE-35508
b67be9c 
* master:
  Bump commons-fileupload:commons-fileupload from 1.5 to 1.6.0 (#409)
  Bump org.apache.commons:commons-lang3 from 3.17.0 to 3.18.0 (#412)
  Bump org.apache.tomcat:tomcat-coyote from 11.0.8 to 11.0.9 (#411)
  Finalize Notification Tab Removal
feat: replace user api w/ user autodeletion feature
3cd8164
feat: add dry run when auto user deletion is enabled
8545734
fix: implement not null for last_login
cda3d3c
fix: removed unneccessary indexes
ecbb8b7
update index to only user + token
1a2e749
updated indexes
25b4e7c
updated custom SQL w/ CriteriaBuilder
1c9d6ed
fix: replace native SQL queries
c5a68b1
Merge branch 'master' into zkofiro/SRE-35508
ce85bed
updated TankConfig for XMLConfig changes
52f0ed0
@Zakaria-Kofiro Zakaria-Kofiro merged commit 04423d4 into master Aug 19, 2025
3 checks passed
@Zakaria-Kofiro Zakaria-Kofiro deleted the zkofiro/SRE-35508 branch August 19, 2025 01:06
kevin-mcgoldrick added a commit that referenced this pull request Aug 20, 2025
@kevin-mcgoldrick
Merge branch 'master' into chartjs
4306686 
* master:
  Implement user auto-deletion follow up PR: Update TankConfig to account for XMLConfig changes (#417)
  Implement user auto-deletion for inactive accounts with configurable retention (default 2+ years) (#413)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevin-mcgoldrick kevin-mcgoldrick kevin-mcgoldrick approved these changes

Assignees

@Zakaria-Kofiro Zakaria-Kofiro

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Zakaria-Kofiro @kevin-mcgoldrick