search

michelp / pgsodium

Modern cryptography for PostgreSQL using libsodium.
Other
546 stars 32 forks source link

Improve startup log messages #45

Closed ioguix closed 1 year ago

ioguix commented 1 year ago

When the getkey_script file is missing, previous startup error message was wrongly showing "Permission denied for %s". This shows a relevant error message with addition of some details and hints.

The patch also improves other startup error/log messages so they repect the PostgreSQL log format if stderr is collected.

michelp commented 1 year ago

Hmm github won't run the tests it says "Expected-- Waiting for status to be reported" that's... unclear. Anyway, googling is giving me vague answers about personal access tokens, any thoughts on what I'm waiting to be reported to from what?

ioguix commented 1 year ago

Hmm github won't run the tests it says "Expected-- Waiting for status to be reported" that's... unclear. Anyway, googling is giving me vague answers about personal access tokens, any thoughts on what I'm waiting to be reported to from what?

I have absolutely no clue... First time I'm seeing this.

I rebased the branch on current HEAD by the way.

ioguix commented 1 year ago

Maybe you can somehow manually trigger the tests workflow?