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

Complete DB tasks to enable merging of admin.storage PR #1592

Closed
3 tasks done
JeffLowe opened this issue Mar 9, 2022 · 1 comment
Closed
3 tasks done

Complete DB tasks to enable merging of admin.storage PR #1592

JeffLowe opened this issue Mar 9, 2022 · 1 comment
Assignees

Comments

@JeffLowe
Copy link
Contributor

JeffLowe commented Mar 9, 2022

admin.storage updates to manage pinning, storage limits, and user blocking nearing completion in this admin.storage PR. https://github.com/nftstorage/admin.storage/pull/22

Before the admin.storage PR can be merged the following needs to be done in the following order:

  • 1. nft.storage DB schema in prod needs the latest user_tag_type and user_tag SQL ( @hugomrdias said he'll be able to have this completed in the next day or so)
  • 2. we need to run INSERT INTO statements in the nft.storage prod DB to migrate the current values of PSA_ALLOW into user_tag with HasPsaAcces=true (i've attached a migration script with values @vasco-santos provided on February 26th. by now there's likely some user_ids that have been added, therefore i need an updated list of user_ids in PSA_ALLOW. i wrote a script to generate this migration file so if someone can get me an updated PSA_ALLOW value from prod nft.storage i can provide a more up to date migration file)
  • 3. approve and merge the PR that migrates nft.storage to use the user_tag table instead of the global PSA_ALLOW cloudflare variable feat: refactor pinning authorization logic to use user_tag table #1389
@hugomrdias
Copy link
Contributor

@trigramdev9 send me the insert statements to finish 2. and 3. is already reviewed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants