Skip to content

Create a global client for OFF API - #314

Merged
littlecoffe8 merged 5 commits into
mainfrom
fix/create-global-client
May 15, 2026
Merged

Create a global client for OFF API#314
littlecoffe8 merged 5 commits into
mainfrom
fix/create-global-client

Conversation

@littlecoffe8

@littlecoffe8 littlecoffe8 commented Apr 16, 2026

Copy link
Copy Markdown
Collaborator

Description

Create a global client for OFF API to avoid "too many connections" answer from OFF

Code changes

  • create http_client.py
  • adapt knowledge_panel.py
  • close connection in main.py
  • update tests

How to test

Checks logs when requesting many products in test page

@littlecoffe8 littlecoffe8 left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure there is not going to be to many requests if we receive a large batch request of knowledge panels

@littlecoffe8 littlecoffe8 changed the title feat/round-pain-durations (#312) Create a global client for OFF API Apr 16, 2026
@littlecoffe8
littlecoffe8 force-pushed the fix/create-global-client branch from 5b3d94d to 36e2824 Compare May 15, 2026 15:02
@littlecoffe8
littlecoffe8 merged commit 003d9a1 into main May 15, 2026
2 checks passed
@littlecoffe8
littlecoffe8 deleted the fix/create-global-client branch June 20, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant