ranch icon indicating copy to clipboard operation
ranch copied to clipboard

Obfuscate private key password in logs

Open juhlig opened this issue 3 years ago • 6 comments

Closes #333

juhlig avatar Dec 07 '21 10:12 juhlig

Hm... the upgrade suite fails. I'll look into that later, it is probably because 2.1 is now the current version or something :\

juhlig avatar Dec 07 '21 11:12 juhlig

Sounds good.

essen avatar Dec 07 '21 11:12 essen

I rebased this PR on #338, and the test pass now for all OTP versions except 25-rc3, where it fails in connection with SSL all over the place. Regardless of this PR, I think this is something that needs to be investigated soon as OTP 25 is just around the corner.

juhlig avatar May 12 '22 13:05 juhlig

Where do you see SSL problems? Looking at the Arch Linux logs for example the problem is that Relx is not compatible with OTP 25 and I have fixed that in an Erlang.mk branch (for now) and have patches locally for Ranch and Cowboy. So don't worry about it I will merge all that in order when OTP 25 is released.

essen avatar May 12 '22 13:05 essen

Where do you see SSL problems?

In Ubuntu, MacOS, Debian, Alpine. All tests there pass, until they get to 25-rc3, where 9 in the acceptors suite and 6 in the proxy header suite fail like this:

{error, {tls_alert, {handshake_failure, "TLS client: In state hello received SERVER ALERT: Fatal - Handshake Failure\n"}}}

juhlig avatar May 12 '22 14:05 juhlig

I see. It has to do with certificate verification on the client side. Probably doesn't like the self-signed certificate. Not sure why it wouldn't fail on Arch Linux.

essen avatar May 12 '22 14:05 essen

Hello! Please rebase and let's ignore the expected macOS failure for now.

essen avatar May 02 '23 09:05 essen

Sorry, this totally fell between the cracks XD Rebased.

juhlig avatar Oct 17 '23 09:10 juhlig

Looks like the buildkite CI is down?

juhlig avatar Oct 19 '23 07:10 juhlig

You've used all of the job minutes included in your Free plan for this month. To continue running your builds, select a different Buildkite plan.

Welp. But I have a complimentary account.

Might be a good time to move to Actions instead of BuildKite.

essen avatar Oct 19 '23 08:10 essen

There's some flaky tests it seems but master is using Actions now, so please rebase so your PR can go through CI. Thanks!

essen avatar Dec 05 '23 10:12 essen

Done. In 26.1.2 on MacOS the upgrade suite fails, but I don't know what the problem could be. Timeout in os:get_data it says, seems unrelated to this PR.

juhlig avatar Dec 06 '23 07:12 juhlig

I think it's a flake I've seen it while preparing actions.

essen avatar Dec 06 '23 08:12 essen

Merged, thanks!

essen avatar Dec 06 '23 10:12 essen