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

add --password to supabase migration fetch #2993

Open
BinaryShrub opened this issue Dec 23, 2024 · 6 comments
Open

add --password to supabase migration fetch #2993

BinaryShrub opened this issue Dec 23, 2024 · 6 comments
Labels
enhancement New feature or request

Comments

@BinaryShrub
Copy link

Almost all other cli commands will accept -p but this one won't :(

Hopefully a quick fix for someone who knows what they are doing?

@sweatybridge
Copy link
Contributor

I agree we should make the flag more consistent.

A temporary workaround is to set SUPABASE_DB_PASSWORD=your-pass env var.

@sweatybridge sweatybridge added the enhancement New feature or request label Dec 30, 2024
@Sanket-0510
Copy link

Hey @sweatybridge I would like to work on this

@sweatybridge
Copy link
Contributor

Hi @Sanket-0510, thanks for stepping up for this task.

This is how the password flag is currently added to commands. It would be great to do the same for other commands which support --db-url flag to add the -p flag.

Feel free to open a PR and tag me if you run into any issues.

@Sanket-0510
Copy link

hey but as the db-url already consist of the database password for eg. - postgresql://postgres:[YOUR-PASSWORD]@<url>/postgres why do we provide an option for the password can't we just extract from the URL incase we want to create configuration for the databse. This something popped out in my mind when I saw the db-url structure.

@sweatybridge
Copy link
Contributor

Yes, using db-url flag is another alternative that's available today. Most of the time, these 3 flags are always present on the same command

  • db-url
  • linked
  • local

Since linking a project also saves the password locally, so the password flag is not necessary.

Perhaps the only place it's useful is when used together with --project-ref or --project-id flag.

@yaten2302
Copy link
Contributor

Hey @sweatybridge , may I work on this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants