Skip to content

Commit

Permalink
STS needs to be statically linked
Browse files Browse the repository at this point in the history
This is needed to support loading roles on kubernetes pod using web identity federation, if STS is not there, the aws-sdk credentials chain will ignore this method and fallback to the instance metadata role which would not be the expected role for the application code
  • Loading branch information
osalloum committed Jun 5, 2024
1 parent 42c78d3 commit f460a86
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ build_loadable_extension(${TARGET_NAME} ${PARAMETERS} ${EXTENSION_SOURCES})
# Weirdly we need to manually to this, otherwise linking against
# ${AWSSDK_LINK_LIBRARIES} fails for some reason
find_package(ZLIB REQUIRED)
find_package(AWSSDK REQUIRED COMPONENTS core)
find_package(AWSSDK REQUIRED COMPONENTS core sts)

# Build static lib
target_include_directories(${EXTENSION_NAME}
Expand Down
7 changes: 5 additions & 2 deletions vcpkg.json
Original file line number Diff line number Diff line change
@@ -1,9 +1,12 @@
{
"dependencies": [
"zlib",
"aws-sdk-cpp",
{
"name": "aws-sdk-cpp",
"features": [ "sts" ]
},
"openssl"
],
"builtin-baseline": "a1a1cbc975abf909a6c8985a6a2b8fe20bbd9bd6",
"overrides": [{"name": "openssl", "version": "3.0.8"}]
}
}

0 comments on commit f460a86

Please sign in to comment.