certbot-he-hook icon indicating copy to clipboard operation
certbot-he-hook copied to clipboard

Fixed HE.net authentication & domain not found error message variable.

Open JeffWDH opened this issue 4 years ago • 3 comments

My domains didn't auto renew and when I went to investigate it seems like Hurricane Electric no longer has "CGISESSID" in their cookie variable. SESSID logins are likely broken but this pull fixes logins using your username and password.

JeffWDH avatar Jun 05 '20 18:06 JeffWDH

For the HE_SESSID , probably this is enough to fix it also:

 elif [ -n "$HE_SESSID" ]; then
-   HE_COOKIE="CGISESSID=${HE_SESSID}"
+   HE_COOKIE="${HE_SESSID}"
 else

And the HE_SESSID must be the full cookie, taken from the cookie after a successful login in the browser... but it will probably work only for about 12hours

danielmotaleite avatar Jun 08 '20 23:06 danielmotaleite

The patch works! 👍 Thanks JeffWDH!

oleg-fiksel avatar Jun 14 '20 16:06 oleg-fiksel

Can confirm this patch allows renewals with user/pass.

May thanks @JeffWDH

simaoafonso-pwt avatar Jun 16 '20 11:06 simaoafonso-pwt