Skip to content

fix: only use base_uri from http client - #481

Merged
Vincent Biret (baywet) merged 2 commits into
mainfrom
fix/http-base-uri-initialization
Jun 26, 2025
Merged

fix: only use base_uri from http client#481
Vincent Biret (baywet) merged 2 commits into
mainfrom
fix/http-base-uri-initialization

Conversation

@gavinbarron

@gavinbarron Gavin Barron (gavinbarron) commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

Overview

Updates logic for initializing kiota httpx request adapter to only use the base_uri from the http client. This aligns with the behavior in the other kiota http request adapters.
This leaves a dangling unused parameter, however removing it would be a breaking change so we're better to leave it for now.

Related Issue

Contributes to fixing: microsoftgraph/msgraph-beta-sdk-python#743

Testing Instructions

  • Expanded tests cases demonstrate how to test.

@baywet

Copy link
Copy Markdown
Member

depends on #483 to be merged first

@sonarqubecloud

Copy link
Copy Markdown

@github-project-automation github-project-automation Bot moved this to In Progress 🚧 in Kiota Jun 26, 2025
@baywet
Vincent Biret (baywet) merged commit b5e2fd1 into main Jun 26, 2025
53 checks passed
@baywet
Vincent Biret (baywet) deleted the fix/http-base-uri-initialization branch June 26, 2025 23:10
@github-project-automation github-project-automation Bot moved this from In Progress 🚧 to Done ✔️ in Kiota Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants