Database errors crash vieter #280

Open
opened 2022-10-30 09:10:09 +01:00 by Jef Roosens · 2 comments
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

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.

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

This'll be fixed by #284.

This'll be fixed by #284.
Jef Roosens added a new dependency 2022-11-01 20:05:12 +01:00
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
Jef Roosens added the
V
label 2022-12-22 20:51:49 +01:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Depends on
#268 Update to V 3.2
vieter-v/vieter
Reference: vieter-v/vieter#280
There is no content yet.