Skip to content

Cosmin/feature/mysql users#111

Merged
CosminBuzicUni merged 17 commits into
developfrom
Cosmin/feature/mysql_users
Jan 17, 2026
Merged

Cosmin/feature/mysql users#111
CosminBuzicUni merged 17 commits into
developfrom
Cosmin/feature/mysql_users

Conversation

@CosminBuzicUni

@CosminBuzicUni CosminBuzicUni commented Jan 17, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@CosminBuzicUni
Added db roles and permission. Explained readme
73c41bb
@CosminBuzicUni
Added testing
375e632
@CosminBuzicUni
Added roles integration into the api endpoints and testing
e5b9a61
@CosminBuzicUni
Refractored TESTING_GUIDE and updated it
81aeb5c
@CosminBuzicUni
Added ownership/sharing validation for vaults crud, credentials crud,…
4197d8f 
… vault sharing, added a lot of logging, audit UserRoleChanged action
@CosminBuzicUni
Credentials fully work now with validation. DB artifacts added
5ff9b7e
@CosminBuzicUni
Added justification in readme for DB constraints/artifacts/isolation
281c603
@CosminBuzicUni
Added backup/recovery document
0d33e1a
@CosminBuzicUni
Added credentials testing
6793757
@CosminBuzicUni
added vault tests
f2676e7
@CosminBuzicUni
VaultSharesTests
c5b1a91
@CosminBuzicUni
fixed sp_AddVaultShare procedure
c68a0df
@CosminBuzicUni
added auth assign roles test
31e7fbf
@CosminBuzicUni
Added complete credentials tests. update and delete were missing
ac93e6a
@CosminBuzicUni
Added misisng vault tests. Update, delete
b1ef26d
@CosminBuzicUni
Added misisng vault shares tests. shared user cannot revoke 403 and n…
9428dfb 
…onexistent user 404
@CosminBuzicUni
Merge branch 'develop' of https://github.com/pop9459/DataProcessing-P…
43ab8b6 
…assMan into Cosmin/feature/mysql_users

# Conflicts:
#	PassManAPI/Controllers/CredentialsController.cs
#	PassManAPI/Controllers/VaultSharesController.cs
#	PassManAPI/Controllers/VaultsController.cs
@pop9459

pop9459 commented Jan 17, 2026

Copy link
Copy Markdown
Owner

Is it okay to merge this @copilot ????

Copilot AI commented Jan 17, 2026

Copy link
Copy Markdown

@pop9459 I've opened a new pull request, #112, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI mentioned this pull request Jan 17, 2026
Closed
@CosminBuzicUni

CosminBuzicUni commented Jan 17, 2026

Copy link
Copy Markdown
Collaborator Author

#88, #95, #96, #98, #89, #94, #99, #100, #90, #93, #92, #102 done

This was linked to issues Jan 17, 2026
Priority 9: Comprehensive API Testing #88
Closed
Priority 10: Database Users and Permissions #89
Closed
Priority 11: Enhanced Data Validation and Error Responses #90
Closed
Implement token expiration and refresh flow #93
Closed
Create Postman collection for API testing #94
Closed
This was linked to issues Jan 17, 2026
Create automated backup scripts #95
Closed
Add [Authorize] attributes to protected endpoints #96
Closed
Document and implement transaction isolation levels #98
Closed
Create Database Triggers for audit logging #99
Closed
Create SQL Views for data aggregation #100
Closed
@CosminBuzicUni CosminBuzicUni merged commit ae4bacb into develop Jan 17, 2026
This was referenced Jan 18, 2026
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment