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

[Bug] Spurious warning on fresh database #249

Open
samcarey opened this issue Nov 4, 2024 · 0 comments
Open

[Bug] Spurious warning on fresh database #249

samcarey opened this issue Nov 4, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@samcarey
Copy link

samcarey commented Nov 4, 2024

Describe the bug

On a fresh startup with no previously existing database, upon a GET, this match case gets matched and prints a warning even though nothing is wrong.

I think it should actually be

(Ok(Some(_)), Ok(None)) => {

Ok(Some(_)) instead of Ok(_)

To reproduce

  1. Enable logging at warning or more verbose.
  2. Start the plugin with no existing database and the create_db key present in the config
  3. Do a GET against the plugin

System info

  • MacOS, M2
@samcarey samcarey added the bug Something isn't working label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant