Skip to content

Address audit findings for PR-158 and PR-162#164

Merged
8ball030 merged 40 commits intofeat/trading_strategy_params_inspection_and_updatingfrom
fix/audit_pr-163
Feb 4, 2026
Merged

Address audit findings for PR-158 and PR-162#164
8ball030 merged 40 commits intofeat/trading_strategy_params_inspection_and_updatingfrom
fix/audit_pr-163

Conversation

@Karrenbelt
Copy link
Collaborator

Addresses issues identified in audit (#163) of #158 and #162.

Changes:

  • Fix HTTP routing bug (.find.endswith)
  • Remove profanity from code comment
  • Implement case-insensitive asset symbol matching
  • Additionally bumps minimum python version requirement to 3.11, as derive_client uses StrEnum, tho incorrectly lists 3.10 as minimum compatible python version in its pyproject.toml

@8ball030 8ball030 merged commit 5bab39a into feat/trading_strategy_params_inspection_and_updating Feb 4, 2026
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