Account base64 representation is calculated with "\n" #473

New Issue

2025-12-29T01:25:53+01:00

adam commented

2025-12-29 01:25:53 +01:00

Originally created by @domrim on GitHub (Apr 17, 2020).

Hi, at first: Thanks for this great piece of software :)

i stumbled upon a small issue with the account urlbase64 creation. the issue is in line 233 and 236 where CAHASH and OLDCAHASH are created:
https://github.com/dehydrated-io/dehydrated/blob/dbb0ef1ce1a93a20fe1cc532adbdf676cb14c3d3/dehydrated#L232-L244

because the CA-String gets echoed with echo ${CA} and not echo -n {CA}" the actual string which gets Base64 encoded is "https://acme-v02.api.letsencrypt.org/directory\n" instead of "https://acme-v02.api.letsencrypt.org/directory" (for the default ${CA} value).

I don't know if this issue is fixable because it would require to rename the account folder for all existing users, but i wanted to let you know of the issue. I also wouldn't see this as a large issue, it just fell on my feet when i wanted to calculate the base64 string within python.

Originally created by @domrim on GitHub (Apr 17, 2020). Hi, at first: Thanks for this great piece of software :) i stumbled upon a small issue with the account urlbase64 creation. the issue is in line 233 and 236 where `CAHASH` and `OLDCAHASH` are created: https://github.com/dehydrated-io/dehydrated/blob/dbb0ef1ce1a93a20fe1cc532adbdf676cb14c3d3/dehydrated#L232-L244 because the CA-String gets echoed with `echo ${CA}` and not `echo -n {CA}"` the actual string which gets Base64 encoded is "https://acme-v02.api.letsencrypt.org/directory\n" instead of "https://acme-v02.api.letsencrypt.org/directory" (for the default `${CA}` value). I don't know if this issue is fixable because it would require to rename the account folder for all existing users, but i wanted to let you know of the issue. I also wouldn't see this as a large issue, it just fell on my feet when i wanted to calculate the base64 string within python.

adam closed this issue

2025-12-29 01:25:53 +01:00

adam commented

2025-12-29 01:25:54 +01:00

@lukas2511 commented on GitHub (Apr 17, 2020):

Yea this is something I'll just leave as is since it's not really disturbing operation, maybe I'll sneak in a little note for developers with the next commit ;) Thanks for reporting!

@lukas2511 commented on GitHub (Apr 17, 2020): Yea this is something I'll just leave as is since it's not really disturbing operation, maybe I'll sneak in a little note for developers with the next commit ;) Thanks for reporting!

adam referenced this issue

2025-12-29 01:29:38 +01:00

[PR #473] [MERGED] Fix globbing of CONFIG_D *.sh files #847

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/dehydrated#473