Skip to content

[SVLS-7704] add support for SSM Parameter API key#924

Merged
litianningdatadog merged 1 commit into
mainfrom
john/add-ssm-parameter-api-key
Nov 19, 2025
Merged

[SVLS-7704] add support for SSM Parameter API key#924
litianningdatadog merged 1 commit into
mainfrom
john/add-ssm-parameter-api-key

Conversation

@jchrostek-dd

Copy link
Copy Markdown
Contributor

Overview

  • Add support for customers storing Datadog API Key in SSM Parameter Store.

Testing

  • Deployed changes and confirmed this work with Parameter Store String and SecureString.

@jchrostek-dd
jchrostek-dd requested a review from a team as a code owner November 12, 2025 15:01
Ok(client) => client,
let mut aws_credentials = AwsCredentials::from_env();

if aws_credentials.aws_secret_access_key.is_empty()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe change it to a method in aws_credentials?

aws_credentials.is_snap_start()

@duncanista duncanista left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM – might need a public docs update

@litianningdatadog
litianningdatadog merged commit 9f655d6 into main Nov 19, 2025
39 checks passed
@litianningdatadog
litianningdatadog deleted the john/add-ssm-parameter-api-key branch November 19, 2025 19:58
lym953 pushed a commit that referenced this pull request Nov 21, 2025
## Overview
* Add support for customers storing Datadog API Key in SSM Parameter
Store.

## Testing
* Deployed changes and confirmed this work with Parameter Store String
and SecureString.
duncanpharvey pushed a commit that referenced this pull request Mar 10, 2026
## Overview
* Add support for customers storing Datadog API Key in SSM Parameter
Store.

## Testing
* Deployed changes and confirmed this work with Parameter Store String
and SecureString.
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