supabase / vault

Extension for storing encrypted secrets in the Vault

Apache License 2.0

163 stars 9 forks source link

Deleting a key from pgsodium that's being used throws a foreign key error #12

Open joshenlim opened 1 year ago

joshenlim commented 1 year ago

Bug report

Describe the bug

Just wondering if it should automatically be cascaded?

michelp commented 1 year ago

I don't think automatic cascading is a good idea, as the delete may be a mistake, the CASCADE option is a safety belt for that reason. Instead I'd suggest the dashboard UX for deleting rows possibly include a CASCADE dialog? The error tells you what would be deleted on cascade (the dashboard may not be showing this level of error detail, which would be unfortunate and should be addressed).