-
Notifications
You must be signed in to change notification settings - Fork 61
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
Improve the invalid master key file
error
#424
Comments
Could I take on this ticket?~ @fluca1978 |
@T1t4m1un Go for it |
Is this issue got fixed!! or other wise can i go for it @jesperpedersen |
@hariharan0905 @T1t4m1un is currently assigned to it... @T1t4m1un Update ? |
Sorry I forget to update it!!! I'm not at the computer recently, I will update as soon as possible |
Il dom 5 mag 2024, 19:11 HARIHARAN KARTHICK ***@***.***> ha
scritto:
@fluca1978 <https://github.com/fluca1978> Is this still this issue is
assign to titanium
yes it is still assigned.
—
… Reply to this email directly, view it on GitHub
<#424 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGGUYEF3FO7U5GIHEYSFTLZAZR5HAVCNFSM6AAAAABFG72PX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUHA4DCNBVGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@hariharan0905 Please, work on one issue at the time so we know what is being worked on |
Ok sure! |
In different points of
main.c
there are errors about the master key file, for example https://github.com/agroal/pgagroal/blob/master/src/main.c#L536.This can be confusing, since the user is not able to understand what is the problematic file: it could be a different file (e.g.,
pgagroal_users.conf
that cannot be decrypted) or the master key file that does not exists. Moreover, all theerrx
calls exit with1
, while a more specific return code is probided intoret
.More information should be reported both to systemd and to the user.
The text was updated successfully, but these errors were encountered: