Skip to content

Python: Add framework name into UserAgent header for bedrock integration#12901

Merged
TaoChenOSU merged 3 commits into
microsoft:mainfrom
0x-fang:add_framework_name_to_user_agent
Sep 3, 2025
Merged

Python: Add framework name into UserAgent header for bedrock integration#12901
TaoChenOSU merged 3 commits into
microsoft:mainfrom
0x-fang:add_framework_name_to_user_agent

Conversation

@0x-fang

@0x-fang 0x-fang commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

This PR adds the framework name into User-Agent header of Bedrock API call, so it can be used to collect the usage.

Description

Updated integration with Amazon Bedrock services to include the framework name semantic-kernel into the User-Agent header field for Bedrock API call for improved client identification. No changes to user-facing features or workflows. Similar change had been made to other framework such as langchain

Contribution Checklist

@0x-fang 0x-fang requested a review from a team as a code owner August 8, 2025 21:33
@moonbox3 moonbox3 added the python Pull requests for the Python Semantic Kernel label Aug 8, 2025
@github-actions github-actions Bot changed the title chore: Add framework name into UserAgent header for bedrock integration Python: chore: Add framework name into UserAgent header for bedrock integration Aug 8, 2025
@0x-fang

0x-fang commented Aug 12, 2025

Copy link
Copy Markdown
Contributor Author

Hi @moonbox3 @dmytrostruk @eavanvalkenburg Could you please kindly help take a look and merge this small change if possible? Thank you very much!

@moonbox3

moonbox3 commented Sep 3, 2025

Copy link
Copy Markdown
Collaborator

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
connectors/ai/bedrock/services
   bedrock_base.py140100% 
TOTAL27113472782% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
3715 22 💤 0 ❌ 0 🔥 1m 42s ⏱️

@TaoChenOSU TaoChenOSU changed the title Python: chore: Add framework name into UserAgent header for bedrock integration Python: Add framework name into UserAgent header for bedrock integration Sep 3, 2025
@TaoChenOSU TaoChenOSU enabled auto-merge September 3, 2025 17:01
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Sep 3, 2025
Merged via the queue into microsoft:main with commit f1aac84 Sep 3, 2025
28 checks passed
@github-project-automation github-project-automation Bot moved this to Sprint: Done in Semantic Kernel Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests for the Python Semantic Kernel

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants