search

jbms / finance-dl

Tools for automatically downloading/scraping personal financial data.
GNU General Public License v2.0
290 stars 37 forks source link

Account retrieval for TD Bank does not work due to ofxclient issue #69 #13

Open philipsd6 opened 5 years ago

philipsd6 commented 5 years ago

I've opened captin411/ofxclient#69 about this issue, and while my PR captin411/ofxclient#70 will resolve the issue for ofxclient, it will somehow need to be managed in finance-dl as well. Once (if?) that PR is accepted, how would we update our finance-dl config to supply that parameter to Institution.accounts()? Or should finance-dl automatically fix it if tdbank is in the ofx url?

philipsd6 commented 5 years ago

I've updated the PR per your suggestion and now if the PR is accepted, there will be no need to update finance-dl to deal with TD Bank's OFX spec violation. This issue should probably remain open as a reference until the upstream issue is resolved.