Database errors crash vieter #280

New issue

Open

opened 2022-10-30 09:10:09 +01:00 by Jef Roosens · 2 comments

Jef Roosens commented

2022-10-30 09:10:09 +01:00

Owner

2022-10-30 08:07:20 [DEBUG] POST /api/v1/logs?target=50&startTime=1667117178&endTime=1667117239&arch=x86_64&exitCode=8 HTTP/1.1
V panic: sqlite.SQLError: bad parameter or other API misuse (8) (INSERT INTO `BuildLog` (`target_id`, `start_time`, `end_time`, `arch`, `exit_code`) VALUES (?1, ?2, ?3, ?4, ?5);)
v hash: ec75860
backtrace_symbols is missing => printing backtraces is not available.
Some libc implementations like musl simply do not provide it.

My server managed to crash because of an SQLite error. While this will be fixed once we update to a newer V version that supports catching these errors (meaning we'd return a 500 or something instead), this is still worrying.

``` 2022-10-30 08:07:20 [DEBUG] POST /api/v1/logs?target=50&startTime=1667117178&endTime=1667117239&arch=x86_64&exitCode=8 HTTP/1.1 V panic: sqlite.SQLError: bad parameter or other API misuse (8) (INSERT INTO `BuildLog` (`target_id`, `start_time`, `end_time`, `arch`, `exit_code`) VALUES (?1, ?2, ?3, ?4, ?5);) v hash: ec75860 backtrace_symbols is missing => printing backtraces is not available. Some libc implementations like musl simply do not provide it. ``` My server managed to crash because of an SQLite error. While this will be fixed once we update to a newer V version that supports catching these errors (meaning we'd return a 500 or something instead), this is still worrying.

Jef Roosens added the

bug

label 2022-10-30 09:10:09 +01:00

Jef Roosens commented

2022-10-30 09:25:37 +01:00

Author

Owner

Yeah this error happened because I incorrectly configured my Docker volume permissions, making the directory read-only. However, it does expose the issue that database errors can currently crash Vieter.

Jef Roosens changed title from ~~sqlite panic~~ to Database errors crash vieter

2022-10-30 09:25:50 +01:00

Jef Roosens commented

2022-11-01 20:04:54 +01:00

Author

Owner

This'll be fixed by #284.

Jef Roosens added a new dependency 2022-11-01 20:05:12 +01:00

#268 Update to V 3.2

Jef Roosens added this to the 0.5.0 milestone 2022-11-01 20:05:20 +01:00

Jef Roosens added this to the (deleted) project 2022-11-02 08:27:21 +01:00

Jef Roosens removed this from the 0.5.0 milestone 2022-12-14 18:50:37 +01:00