Skip to content
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.

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

Added trailing slash to parameter store examples #1086

Merged
merged 1 commit into from
Mar 10, 2024

Conversation

thelateperseus
Copy link
Contributor

@thelateperseus thelateperseus commented Mar 10, 2024

The trailing slash is required to avoid a leading dot in the property name. For example, if the parameter is named /config/spring/message and the spring.config.import property is aws-parameterstore:/config/spring, then the resulting property in the Spring environment will be .message. Instead, the property should be message, so the spring.config.import property should be aws-parameterstore:/config/spring/

Fixes #1058, #923

The trailing slash is required to avoid a leading dot in the property name. For example, if the parameter is named `/config/spring/message` and the `spring.config.import` property is `aws-parameterstore:/config/spring`, then the resulting property in the Spring environment will be `.message`. Instead, the property should be `message`, so the `spring.config.import` property should be `aws-parameterstore:/config/spring/`

Fixes awspring#1058
@github-actions github-actions bot added the type: documentation Documentation or Samples related issue label Mar 10, 2024
@maciejwalkowiak maciejwalkowiak added the component: parameter-store Parameter Store integration related issue label Mar 10, 2024
@maciejwalkowiak maciejwalkowiak added this to the 3.1.1 milestone Mar 10, 2024
Copy link
Contributor

@maciejwalkowiak maciejwalkowiak left a comment

Choose a reason for hiding this comment

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

Thanks a lot @thelateperseus!

@maciejwalkowiak maciejwalkowiak merged commit 165de38 into awspring:main Mar 10, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: parameter-store Parameter Store integration related issue type: documentation Documentation or Samples related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parameter Store path requires trailing slash
2 participants