-
Notifications
You must be signed in to change notification settings - Fork 18
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
Unable to login on node.js #44
Comments
Hello, HTTPS is required for OAuth2 authentication. There's instructions in the readme on how to generate a self signed SSL cert. Also, both the local config and the TD App in your TD dev account should point to the same address. Hope this helps! |
Hi @knicola , thanks for the reply.
Is this "SSL unsafe legacy renegotiation disabled" error something I can resolve by generating the certificate with a different openssl command? Thanks! |
Hey, can you try running it with node v16? |
Yes, that worked. Thanks for your help & creating this project! |
Hi, I am having trouble getting the login flow to work on node.js.
I copied the sample code you shared in a comment on issue #19, using a locally generated certificate/private key + API_KEY. Everything compiles and runs.
When the login() routine is invoked, it opens up my default web browser to the TD Ameritrade login page and 1. asks for my account credentials and 2. asks if I want to grant my app permission to access to my account. After a successful auth / grant, the browser is redirected to my REDIRECT_URI (e.g. https://localhost/?code=.....) but the node.js app just sits there. I tried closing the browser tab and the entire browser, but the node.js app just sits there, apparently not receiving back what it needs from the browser.
What am I missing here?
Please note I am using Windows 10, I am not sure if this matters for how your library handles this login process. The redirect also appears to be changing my REDIRECT_URI from HTTP to HTTPS so it fails to actually load my local app. Not sure if either of these factors are relevant here but I thought I'd mention both.
Thanks in advance, I'm looking forward to resolving these issues and working with this library.
The text was updated successfully, but these errors were encountered: