Skip to content

fix(reporting): identify account facts HTTP client - #556

Merged
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-account-facts-client-identity-20260930
Sep 30, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
codex/ibkr-account-facts-client-identity-20260930

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

Set a fixed, truthful service User-Agent (QSL-IBKR-AccountFacts/1.0) on the normal account-facts publisher and the isolated empty-body diagnostic. Add assertions on both actual request objects. Only two files, five added lines.

Evidence and validation

  • Existing approved cloud environment, unauthenticated GET to the same endpoint: default Python client returned HTTP 403 / Cloudflare 1010; changing only User-Agent to this service identity returned application HTTP 404 / JSON not_found.
  • 29 targeted tests passed; Ruff and git diff --check passed.
  • The cloud GET comparison establishes the client-identity correction, not authenticated POST or funds-sync success.

Scope

User approved publication/adoption of this correction and one further fixed empty-body diagnostic via the existing isolated workflow. URL, token, body, 15-second timeout, no retry/redirect, 15-minute report freshness and all account guards are unchanged. No Cloudflare security configuration, credential update, broker sampling or real funds/report replay is authorized here.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 8958ce1 into main Sep 30, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/ibkr-account-facts-client-identity-20260930 branch September 30, 2026 09:32
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.

1 participant