Skip to content

Use correct URL - #115

Open
debegr92 wants to merge 1 commit into
csingley:masterfrom
debegr92:url_fix
Open

Use correct URL#115
debegr92 wants to merge 1 commit into
csingley:masterfrom
debegr92:url_fix

Conversation

@debegr92

@debegr92 debegr92 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Fixed the statement URL and ignored the received URL from the generated report.

@debegr92

Copy link
Copy Markdown
Contributor Author

@csingley mind merging this PR, as well as #114 ?
Would really appreciate merge and release on this repository.

@csingley

csingley commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Thanks for the PR. Before merging, could you share more detail on the failure you observed?

Our history shows the current GetStatement host (gdcdyn.interactivebrokers.com/AccountManagement/FlexWebService/GetStatement) was deliberately pinned there and verified against a live token (see 86beba8). This PR switches it back to ndcdyn and also drops the fallback to stmt_access.Url from the response entirely.

Can you share:

  • The exact error/response you got from gdcdyn (status code, error code/message, or raw response body)?
  • Whether this is region/account-specific, since IB's endpoints seem to shift between hosts over time?

Want to make sure this doesn't regress the working config for other users.

@debegr92

debegr92 commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

This PR switches it back to ndcdyn and also drops the fallback to stmt_access.Url from the response entirely.

I think it makes sense to support both, the old URL for version 2 and use the new URL for version 3. I added more information in the issue #112 (comment)

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 this pull request may close these issues.

2 participants