[GH-ISSUE #450] Invalid error handling of non-existent container #137

Closed
opened 2026-10-13 20:13:31 +00:00 by bsakaguchi · 3 comments
Owner

Originally created by @useribs on GitHub (Dec 2, 2023).
Original GitHub issue: https://github.com/amnezia-vpn/amnezia-client/issues/450

github.com/amnezia-vpn/amnezia-client@08c506a489/client/core/servercontroller.cpp (L166C28-L166C28)

it expects "Error: No such container:"

I am not familiar with C, but this error message handling seems wrong, on my system i get the following error when trying to deal with non-existent container:

Error response from daemon: No such container: blad

# docker -v
Docker version 24.0.7, build afdd53b

Proposed fix: expect only No such container: without Error string.

Originally created by @useribs on GitHub (Dec 2, 2023). Original GitHub issue: https://github.com/amnezia-vpn/amnezia-client/issues/450 https://github.com/amnezia-vpn/amnezia-client/blob/08c506a48933e46eaeb105a2b3ae52237c547cd5/client/core/servercontroller.cpp#L166C28-L166C28 it expects "**Error:** No such container:" I am not familiar with C, but this error message handling seems wrong, on my system i get the following error when trying to deal with non-existent container: ``` Error response from daemon: No such container: blad # docker -v Docker version 24.0.7, build afdd53b ``` Proposed fix: expect only **No such container:** without Error string.
Author
Owner

@useribs commented on GitHub (Dec 29, 2023):

Should be fixed #by:
https://github.com/amnezia-vpn/amnezia-client/pull/482

<!-- gh-comment-id:1872155387 --> @useribs commented on GitHub (Dec 29, 2023): Should be fixed #by: https://github.com/amnezia-vpn/amnezia-client/pull/482
Author
Owner

@useribs commented on GitHub (Jan 10, 2024):

fix made it into dev?

<!-- gh-comment-id:1885382283 --> @useribs commented on GitHub (Jan 10, 2024): fix made it into dev?
Author
Owner

@lunardunno commented on GitHub (Jan 10, 2024):

yes.

<!-- gh-comment-id:1885563102 --> @lunardunno commented on GitHub (Jan 10, 2024): yes.
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
amnezia-vpn/amnezia-client#137
No description provided.