Skip to content

Implement resiliency and security recommendations for SDKs #19

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.

Sign up for GitHub

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

Jump to bottom
Open
diaryofamademan wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Implement resiliency and security recommendations for SDKs #19

diaryofamademan wants to merge 1 commit into from

Conversation

@diaryofamademan
Copy link
Contributor

@diaryofamademan diaryofamademan commented Oct 16, 2025

Summary

  • add a resilient HTTP session with retries/timeouts, structured errors, and smarter pagination to the Python SDK while making pandas optional
  • require API keys via environment variables, document development setup, and add lightweight dev dependency metadata with a requirements file and gitignore updates
  • harden the JavaScript SDK by introducing a configurable axios client with retries, removing the hard-coded API key, exposing custom errors, and trimming runtime dependencies

Testing

  • pytest (fails: ModuleNotFoundError: No module named 'requests')

https://chatgpt.com/codex/tasks/task_e_68f07ab35ad4832e9fba207d60e0cbb0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chickenopoulos chickenopoulos Awaiting requested review from chickenopoulos

@suleman1220 suleman1220 Awaiting requested review from suleman1220

Assignees

No one assigned

Labels

codex

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@diaryofamademan