Automatic login to Treehouse script causes recursive loop

Question

Please see my QA session here:
https://stackoverflow.com/questions/45220907/automatic-login-userscript-causes-an-recursive-loop
@Steven Parker (https://teamtreehouse.com/stevenparker)

Ben Os · Accepted Answer

I published a solution in the question there in StackOverflow.
It turns out I used a unfitted class (instead a combination of two classes) to initiate logging in for a non connected user.
This is the correct code for automatic login to Treehouse (as long as Treehouse doesn't keep a user logged in):
```Javascript
if ( document.documentElement.classList.contains("visitor") && document.body.id != "signin" ) {
    window.location.href = "https://teamtreehouse.com/signin";
}
if (window.location.href.match("https://teamtreehouse.com/signin")) {
    setTimeout (function() {
        document.querySelector("button[type=submit]").click();
    }, 2000);
}
```

Steven Parker · Answer

I don't know what browser or plug-in you are using, but assuming that script gets re-executed on each page load, it would click the first submit control on the page 1 second later.  And if that should cause the same page to be re-loaded, well that would explain the loop.
You might need to enclose that setTimeout in a conditional to prevent it from being set again.

Welcome to the Treehouse Community

Looking to learn something new?

Ben Os

Ben Os

Automatic login to Treehouse script causes recursive loop

2 Answers

Ben Os

Ben Os

Steven Parker

Steven Parker