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

Password required when connecting to MySQL Database #35

Open
nicolasmnl opened this issue Sep 12, 2024 · 6 comments
Open

Password required when connecting to MySQL Database #35

nicolasmnl opened this issue Sep 12, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@nicolasmnl
Copy link

Hi guys! First of all, congratulations on becoming open-source!

Second of all, i don't think the password should be required when connecting to a data source, specially locally :) Maybe optional in the case you don't have a password for your local db instance

Let me know what you guys think!

PS: The problem occurred when I tried to connect to my local MySQL Database using Data sources and it kept asking the password even though I don't have one for a simple test instance I use

@lucasfcosta
Copy link
Member

Hey @nicolasmnl! Thanks so much for this issue.

I like this idea. I think we could maybe have a toggle there on that screen that disables the password field and add a tooltip saying that we don't recommend using this setting in production.

Do you want to tackle this issue?

I think we should start with a screenshot of the current form and a quick indication/exploration of where that toggle should go.

@lucasfcosta lucasfcosta added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Sep 13, 2024
@douglasmnegri
Copy link
Contributor

Hi! I'd like to work on this issue. Could you please assign it to me?

@nicolasmnl
Copy link
Author

Hey @nicolasmnl! Thanks so much for this issue.

I like this idea. I think we could maybe have a toggle there on that screen that disables the password field and add a tooltip saying that we don't recommend using this setting in production.

Do you want to tackle this issue?

I think we should start with a screenshot of the current form and a quick indication/exploration of where that toggle should go.

Hi, Lucas! Yes, I would love to :)

Yep, I think that's a good start too! I'll take a look when I can!

@nicolasmnl
Copy link
Author

Hey @nicolasmnl! Thanks so much for this issue.

I like this idea. I think we could maybe have a toggle there on that screen that disables the password field and add a tooltip saying that we don't recommend using this setting in production.

Do you want to tackle this issue?

I think we should start with a screenshot of the current form and a quick indication/exploration of where that toggle should go.

Hi, Lucas! Yes, I would love to :)

Yep, I think that's a good start too! I'll take a look when I can!

Oh, I just saw Douglas' comment! Don't know why i didn't see it at first! So sorry!

I think he can work on that! He'll probably do a better job than me 😁

@lucasfcosta
Copy link
Member

lucasfcosta commented Sep 17, 2024

Thanks for the collaborative attitude on this one @nicolasmnl - we all appreciate your report and willingness to help 💚

I'm sure @douglasmnegri will help out on this one! Just assigned him the issue!

@vieiralucas
Copy link
Member

Hi @douglasmnegri do you still wish to work on this one? Let me know if you don't have the time anymore or if you want some help. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants