pgagroal icon indicating copy to clipboard operation
pgagroal copied to clipboard

Improve the `invalid master key file` error

Open fluca1978 opened this issue 11 months ago • 8 comments

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 the errx calls exit with 1, while a more specific return code is probided into ret. More information should be reported both to systemd and to the user.

fluca1978 avatar Mar 25 '24 13:03 fluca1978

Could I take on this ticket?~ @fluca1978

T1t4m1un avatar Apr 19 '24 10:04 T1t4m1un

@T1t4m1un Go for it

jesperpedersen avatar Apr 19 '24 13:04 jesperpedersen

Is this issue got fixed!! or other wise can i go for it @jesperpedersen

hariharan0905 avatar May 04 '24 19:05 hariharan0905

@hariharan0905 @T1t4m1un is currently assigned to it...

@T1t4m1un Update ?

jesperpedersen avatar May 04 '24 19:05 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

T1t4m1un avatar May 05 '24 10:05 T1t4m1un

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 https://github.com/agroal/pgagroal/issues/424#issuecomment-2094881450, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGGUYEF3FO7U5GIHEYSFTLZAZR5HAVCNFSM6AAAAABFG72PX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUHA4DCNBVGA . You are receiving this because you were mentioned.Message ID: @.***>

fluca1978 avatar May 07 '24 14:05 fluca1978

@hariharan0905 Please, work on one issue at the time so we know what is being worked on

jesperpedersen avatar May 07 '24 15:05 jesperpedersen

Ok sure!

hariharan0905 avatar May 07 '24 15:05 hariharan0905