Skip to content

feat: added integration platform header - #46

Merged
daveomri merged 1 commit into
mainfrom
feat/26-langchain-send-the-platform-header
Oct 21, 2026
Merged

daveomri merged 1 commit into
mainfrom
feat/26-langchain-send-the-platform-header

Conversation

@daveomri

Copy link
Copy Markdown
Collaborator

Runs from this package had no attribution in Apify Console. The connector only appended ; Origin/langchain to the User-Agent. The platform reads the x-apify-integration-platform header.

Set in _create_apify_client, the single place every client is built. Verified live: with the header a run gets integrationTracking.platform = "langchain", without it the field is null. Tests cover both sync and async clients.

Console will keep showing origin API until langchain is registered on the Apify side.

@daveomri daveomri self-assigned this Oct 19, 2026
@daveomri
daveomri marked this pull request as ready for review October 19, 2026 13:01
@daveomri
daveomri requested review from drobnikj and vdusek October 19, 2026 13:02

@vdusek vdusek left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Btw, the next release of the Apify Python API client adds a public user_agent_suffix option for appending a custom suffix to the User-Agent header (see apify/apify-client-python#1145). It should make this change more "ergonomic". It's up to you whether you adopt it or not, just letting you know.

@drobnikj drobnikj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

馃憤

@daveomri
daveomri merged commit 0bb491f into main Oct 21, 2026
10 checks passed
@daveomri
daveomri deleted the feat/26-langchain-send-the-platform-header branch October 21, 2026 11:47
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.

4 participants