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

dim.db no longer writable after creating admin account #608

Open
donmor opened this issue Mar 5, 2024 · 6 comments
Open

dim.db no longer writable after creating admin account #608

donmor opened this issue Mar 5, 2024 · 6 comments
Labels
S-needs-triage Status: Needs triage T-bug Something isn't working the way it is supposed to

Comments

@donmor
Copy link

donmor commented Mar 5, 2024

Describe the bug
dim.db get locked and was no longer writable after creating admin account.

To Reproduce
Steps to reproduce the behavior:

  1. Build dim from latest source (patched by Bump sqlx to 0.7.3 #597)
  2. Run dim and create the admin account
  3. Add a library
  4. See ERROR request{method=POST uri=/api/v1/library version=HTTP/1.1}: dim_web::routes::library: Error committing transaction err=Database(SqliteError { code: 5, message: "database is locked" }) in console

Device and browser including versions:
Chromium 102.0.5005.63 (on Linux / loongarch64 PC)

Additional context
Any operation writing the database gets a 500 response; creating accounts gets 401.
I built dim with this fork(#597), this may affect the behavior of program.

@donmor donmor added S-needs-triage Status: Needs triage T-bug Something isn't working the way it is supposed to labels Mar 5, 2024
@vgarleanu
Copy link
Member

Does the lock occur on the latest master? First time seeing this issue and the scanner/dB code hasn't been modified in a long time.

@donmor
Copy link
Author

donmor commented Mar 5, 2024

Not sure... Now trying #593 to see if it works

@vgarleanu
Copy link
Member

Not sure... Now trying #593 to see if it works

Please try latest master

@donmor
Copy link
Author

donmor commented Mar 5, 2024

I'm using loongarch64 (yet another new risc architecture) and latest master failed to build here because of the aged ring 0.16.20 dep (⊙_⊙;) So I have to use code from PRs with sqlx bumped to 0.7.x and patch it again to use nix 0.27+.

@donmor
Copy link
Author

donmor commented Mar 6, 2024

Okay it IS sqlx 0.7 patches that breaks the db XD
Moving to #593 and #597

@donmor
Copy link
Author

donmor commented Mar 6, 2024

BTW I commented out use crate::inspect::ResultExt; in dim-core/src/scanner/movie.rs and dim-core/src/scanner/tv_show.rs as they throw errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-needs-triage Status: Needs triage T-bug Something isn't working the way it is supposed to
Projects
None yet
Development

No branches or pull requests

2 participants