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

Cannot download from Vanguard #46

Closed
rkhwaja opened this issue Jul 22, 2017 · 12 comments · Fixed by #47
Closed

Cannot download from Vanguard #46

rkhwaja opened this issue Jul 22, 2017 · 12 comments · Fixed by #47

Comments

@rkhwaja
Copy link
Contributor

rkhwaja commented Jul 22, 2017

Vanguard is returning an empty string as the body. Since around 7/19/2016, I think.

@jantman
Copy link
Contributor

jantman commented Jul 30, 2017

I can confirm this for me as well, since July 21, 2017 (I think @rkhwaja had the year wrong in his comment).

I run via cron every morning around 2AM Eastern. Vanguard had worked reliably since March through July 20th. From the 21st on, it's getting 0-byte responses.

@jantman
Copy link
Contributor

jantman commented Jul 30, 2017

Ok, I spent about an hour trying various things. The OFX statements that I manually download from Vanguard's site are still OFX version 102, but they include "INTU.USERID" and "INTU.BID" (FITID) fields in the signon, and have the userid in all uppercase. I tried changing the signon to those fields, adding the INTU.BID field, and various different APPVERs for QWIN (2200, 2300, 2400, 2500). I haven't been able to find anything that works. Sometime this week I'll call their tech support and see if I can get any usable information...

@rkhwaja
Copy link
Contributor Author

rkhwaja commented Jul 30, 2017

Can anyone still download Vanguard ofxs? Maybe it's something to do with the accounts themselves. For instance, I haven't made the switch from mutual fund to brokerage accounts yet and my downloads don't work.

@MarkLodato
Copy link

MarkLodato commented Jul 31, 2017

Same here - Vanguard stopped working just as @jantman said. My other bank (PNC) still works fine, so it seems like some change on Vanguard's end.

@rkhwaja
Copy link
Contributor Author

rkhwaja commented Jul 31, 2017

If I download a new ofx file from the website, my account numbers have the prefix "0540-", whereas I've been using account numbers without that prefix. However, I tried adding the prefix when downloading through ofxclient but it still didn't download a valid ofx.

@jantman
Copy link
Contributor

jantman commented Jul 31, 2017

I found a comment about the PocketSense software, which from what I can tell has a lot of code in common with ofxclient (I believe they're both descended from the same code, or something like that) confirming the Vanguard issue with that code as well.

The author of that code said that he figured out a workaround to fix Vanguard, and the problem seems to be that Vanguard has introduced and is now requiring the use of session cookies, which I'm not sure is really compliant in the OFX spec, but at least the author and one user of that package has confirmed the fix. See this portion of the diff for the fix they used; it seems to try the request once, and if it fails, it resends it with cookies returned in the failure message.

I'm going to try a similar fix in my fork and see if it works.

@jantman
Copy link
Contributor

jantman commented Jul 31, 2017

Ok, I have a quick fix that works. I'm working on getting it polished up and pushed up in a branch and PR now.

@jantman
Copy link
Contributor

jantman commented Jul 31, 2017

I have a PR open ( #47 ) with a fix that works for me, from the vanguard-fix branch of my fork. If anyone would like to give it a try, you can install from that branch with: pip install git+https://github.com/jantman/ofxclient.git@vanguard-fix#egg=ofxclient

@rkhwaja
Copy link
Contributor Author

rkhwaja commented Aug 1, 2017

Works for me - thanks. I didn't have to make any changes to my login details.

@scanlom
Copy link

scanlom commented Aug 4, 2017

Good stuff, works now - had to use caps for my username

@captin411
Copy link
Owner

captin411 commented Aug 4, 2017 via email

@MarkLodato
Copy link

Friendly ping. Any chance to merge this fix? Thanks!

captin411 added a commit that referenced this issue Jan 8, 2018
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 a pull request may close this issue.

5 participants