Skip to content

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

Open
JeffWDH wants to merge 2 commits intoangel333:masterfrom
JeffWDH:auth_fix
Open

Fixed HE.net authentication & domain not found error message variable.#6
JeffWDH wants to merge 2 commits intoangel333:masterfrom
JeffWDH:auth_fix

Conversation

@JeffWDH
Copy link
Copy Markdown

@JeffWDH JeffWDH commented Jun 5, 2020

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.

@danielmotaleite
Copy link
Copy Markdown

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

@oleg-fiksel
Copy link
Copy Markdown

oleg-fiksel commented Jun 14, 2020

The patch works! 👍
Thanks JeffWDH!

@simaoafonso-pwt
Copy link
Copy Markdown

Can confirm this patch allows renewals with user/pass.

May thanks @JeffWDH

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants