fix: enhance wallet deletion validation and add null check for orgId in schema and credential-definition - Githubissues

credebl / platform

Open source, Open standards based Decentralised Identity & Verifiable Credentials Platform

Apache License 2.0

31 stars 13 forks source link

fix: enhance wallet deletion validation and add null check for orgId in schema and credential-definition #791

Closed KulkarniShashank closed 2 weeks ago

KulkarniShashank commented 2 weeks ago

What

Improved the validation logic for deleting a wallet, ensuring that references in related tables are properly checked.
Added validation to ensure orgId is null or not in schema and credential definition entities.

Why

To prevent accidental deletion of wallets that are still referenced in other parts of the database, which could lead to data integrity issues.
To ensure that orgId is always present in schema and credential definition entities, null reference as an optional.

How

Implemented additional validation checks before allowing the deletion of a wallet, ensuring no existing references are left in related tables.
Added validation rules in the schema and credential definition code to enforce that orgId null as an optional

sonarcloud[bot] commented 2 weeks ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
1.1% Duplication on New Code

See analysis details on SonarCloud