Skip to content

dev-cpp/azure-identity: Sync with Gentoo to fix managed identity issue #3185

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

Merged
merged 1 commit into from
Aug 14, 2025
+28 −2
Merged

dev-cpp/azure-identity: Sync with Gentoo to fix managed identity issue #3185

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sdk_container/src/third_party/portage-stable/dev-cpp/azure-identity/Manifest
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
DIST azure-sdk-for-cpp-de3cc64a55b2a67d672b7ca899a8675182d1c989.tar.gz 3359660 BLAKE2B eda2034cc789ad6d5e1d477bbb5a3619a53ee22b511707b8aec9471dc8b09814b4b6c199442532790f7b43c07e21bfa3f6534cf49e1f46d7f8a5e60f3e5eecce SHA512 b6004de0e1f312bdd55f91f597d2ee831be9d3d500a0c554a8258f52d054c288a6caff3ae86cc023b9ee1167df9fd898d65278f0b076ae23b2e65e9ab5791d36
DIST azure-sdk-for-cpp-6aea93d0410f5bc3e3a58a8d492a3063cac3aced.tar.gz 3360426 BLAKE2B 027729aece30196a607a3bb9d6b12e1f884069526b96773701fe211945426a0175ed94d65ae5df6577dfe871a89cbbcff53ea1c6ab814719385af9c9a821aad5 SHA512 f7abcf97468caa04c841935911490c134f1d860b9ae11f2ba57a2a480b9c63a7277fe2ed1440ae0d3c8d2ba76890bc170f38ee936016f9a4ba9b4989f2b7eb11
DIST azure-sdk-for-cpp-e8d34efc671b7dff133bd32bd2fe65aab4737ef2.tar.gz 3330505 BLAKE2B 3693cc1d587ad3cba32f5dfdbbc650fbd077f05092f0bd44809d1b99b27a2e3e4a9eeb6fa725e5385f117cd76fb09101839e74c3fdc1e082b45d824cad059fb4 SHA512 ffa0ec9e10dfefd948607761dcc328d39bbb3ed56aa9d1df7c1e370fcfd61e91df6f207c85a97e84fb1bd5ad7f2ec848f2f5f1cb85181174c6c7850ade265eeb
6 changes: 5 additions & 1 deletion ...ure-identity/azure-identity-1.12.0.ebuild → ...-identity/azure-identity-1.13.0-r1.ebuild
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ EAPI=8

inherit cmake

COMMIT="de3cc64a55b2a67d672b7ca899a8675182d1c989"
COMMIT="6aea93d0410f5bc3e3a58a8d492a3063cac3aced"
MY_P="azure-sdk-for-cpp-${COMMIT}"
DESCRIPTION="Azure SDK for C++"
HOMEPAGE="https://azure.github.io/azure-sdk-for-cpp/"
Expand All @@ -30,6 +30,10 @@ BDEPEND="
doc? ( app-text/doxygen )
"

PATCHES=(
"${FILESDIR}"/${PN}-imds-double-slash.patch
)

src_configure() {
local mycmakeargs=(
-DBUILD_DOCUMENTATION=$(usex doc)
Expand Down
22 changes: 22 additions & 0 deletions ..._party/portage-stable/dev-cpp/azure-identity/files/azure-identity-imds-double-slash.patch
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
From 72f327b2c19afffa9df2ce0ef817c13b3dd75804 Mon Sep 17 00:00:00 2001
From: James Le Cuirot <[email protected]>
Date: Wed, 13 Aug 2025 15:52:31 +0100
Subject: [PATCH] Fix IMDS token requests for managed identities

This was broken by a035ee5f9416ef9188533de40b4ab1c37fb7c0af, which
accidentally constructed the IMDS URL with a double slash at the start
of the path. This is not properly routed on the server side, leading to
a 404 error with some very misleading XML.

Signed-off-by: James Le Cuirot <[email protected]>
--- a/src/managed_identity_source.cpp
+++ b/src/managed_identity_source.cpp
@@ -527,7 +527,7 @@ std::unique_ptr<ManagedIdentitySource> ImdsManagedIdentitySource::Create(

imdsUrl = Core::Url{imdsEndpointEnvVarValue};
}
- imdsUrl.SetPath("/metadata/identity/oauth2/token");
+ imdsUrl.SetPath("metadata/identity/oauth2/token");

return std::unique_ptr<ManagedIdentitySource>(
new ImdsManagedIdentitySource(clientId, objectId, resourceId, imdsUrl, options));