Skip to content
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

Pruning short-lived credentials #3001

Open
reinkrul opened this issue Mar 29, 2024 · 1 comment
Open

Pruning short-lived credentials #3001

reinkrul opened this issue Mar 29, 2024 · 1 comment

Comments

@reinkrul
Copy link
Member

reinkrul commented Mar 29, 2024

With OpenID4VP, the Authorization Server will issue an EmployeeCredential (to the user's session wallet) for each initiated OpenID4VP flow. These will all be stored in the issuer's SQL database, never cleaned up.

Potential solutions:

  • An apparent simple solution would be to introduce a cleanup-job for the issuer store, but when the Nuts node is used as primary VC issuer (e.g. CiBG), automated cleanup of issued credentials might destroy administration? Might also break statuslist2021?
  • Alternatively, we could introduce a "store in issuer administration" flag, but that feels like hacky design (and leads to awkward APIs).
  • Automatically clean up short-lived credentials
  • ...

The requirements question behind it is; when would an issuer be cleaning up issued VCs?

@woutslakhorst
Copy link
Member

These particular credentials do not have a CredentialStatus entry so they are non-revocable. Therefore there's no use in storing them in the issuer DB. They are also not issued via an API but via an internal interface. Primary use of the issuer DB is to search for revocable VCs.

For auditing purposes I would store them at the usage side, not the issuing side.

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