-
Notifications
You must be signed in to change notification settings - Fork 5
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
Cannot authenticate #1
Comments
Dear Dylan,
It appears that TDA has changed the nature of their login authentication between the last time I ran a script and now. I run strategies with the plug-in that do not require it to execute continually. I tried running the same script you did and could not log-in. But I ran it with a higher VERBOSITY setting. Here’s what I got:
I’m on another project at the moment, so may not get around to looking deeper into this for a while. But the complete source code is available on GitHub. Feel free to pull it into Visual Studio and run it with breakpoints in the Broker Login logic, which may help determine what has changed.
Clyde Ford
From: Dylan Mozlowski ***@***.***>
Sent: Sunday, March 21, 2021 8:37 AM
To: cwford/TDAmTrade_Zorro_Plugin ***@***.***>
Cc: Subscribed ***@***.***>
Subject: [cwford/TDAmTrade_Zorro_Plugin] Cannot authenticate (#1)
I followed the directions on the main page of the wiki:
* Moved the three dll's to the Plugin folder
* Create a tda developer account and created an app with the callback URL set to http://127.0.0.1
* Added a tda.json to the Data folder:
{
"currency" : "USD",
"langResx" : "en-US",
"tdaAccountNum" : "<my_account_num>",
"testAssets": "GRPN,HUSA,LLEX,ZOM"
}
* Opened Zorro, set the account to Demo and the broker to TD Ameritrade
* Set the Login field to my consumer key and left the password field empty
* Selected Script Workshop1 (also tried other ones such as Z8)
Press Trade
Result:
<https://user-images.githubusercontent.com/4955191/111910882-669bb880-8a20-11eb-9cc7-d110b465c05a.png>
Any ideas or ways I can get more debug/log information?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#1> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABDTWACQM3AGNKV7I46H7MDTEYHCVANCNFSM4ZRVMBVA> . <https://github.com/notifications/beacon/ABDTWAGGJZ3TQGCWN55I3F3TEYHCVA5CNFSM4ZRVMBVKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4MPFJBMQ.gif>
|
I got the plugin to successfully authenticate with TD Ameritrade, and it completed the tests. First, the user needs to ensure that they have recently followed the steps outlined here: I had done the above steps previously, but doing it again helped. There was one more issue, though, which required me to make a change to the source code and recompile the plugin. I will see about doing a pull request for this. |
Has anyone else experienced issues with the contractUpdate (brokerCommand is GET_CONTRACTS)? I know it's not listed on the wiki, but I was playing around with it, and found that the CONTRACT structure in Zorro did not match the one in the TDA plugn code. I modified my local TDA code to use int (32-bit) for Expiry and Type, instead of long (64-bit), since Zorro has them as 32-bit integers. After a couple of other mods to accommodate those changes, everything works beautifully in my Zorro installation. Maybe I did something wrong in the build, but this is working for me. I'm new to GitHub (at least for personal accounts) and somewhat to C#, so please forgive me if I've gotten something wrong. |
@jalbright015 could you build the dlls and release them into your fork? I tried building with your code additions in the PR but still no luck. |
I followed the directions on the main page of the wiki:
http://127.0.0.1
Press Trade
Result:
Any ideas or ways I can get more debug/log information?
The text was updated successfully, but these errors were encountered: