Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cookie expiration check reversed #14

Open
enki opened this issue Dec 13, 2011 · 1 comment
Open

cookie expiration check reversed #14

enki opened this issue Dec 13, 2011 · 1 comment

Comments

@enki
Copy link

enki commented Dec 13, 2011

    var now = new Date();
    var expires_time = parseInt(facebook_cookie['expires'], 10) * 1000;

    if (now.getTime() < expires_time)
    {
        /*
         * The token is expired.
         */
        cb();
        return ;
    }

makes no sense - the cookie is supposed to expire in the future (replace < with >)
this breaks for me with run_example.js

@DracoBlue
Copy link
Owner

Hmm, this means that the expires time conversion must be broken, too. Because for me it worked even with this very wrong line of code. Will look into that.

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

No branches or pull requests

2 participants