AWS Parameter Store with Hoxton SR8+ doesn't override properties when using multiple profiles

[peteguyatt]

Type: Bug

Component:

Parameter Store

Describe the bug
When using multiple active profiles with spring cloud Hoxton SR10 and boot 2.3.9 we have noticed that property order seems to have changed, for example if we have two properties set E.g.

spring.profiles.active=profile_a,profile_b

Parameter Store configured as:

application/profile_a/app.secret-key: abc-123
application/profile_b/app.secret-key: efd-213

and in our bean we have

@Value("${app.secret-key}")
private String secretKey; 

the value resolved will be abc-123 instead of efd-213

I looked through the commit history for AwsParamStorePropertySourceLocator and the issue seems to have been introduced in 582fdc6 where the Collections.reverse has been removed due to switching to using a LinkedHashSet

This looks like an accidental bug to me and the same bug exists in master and the 2.3.x branch

[maciejwalkowiak]

@MatejNedic can you take a look at this one? We should fix it in both 2.2.x in this repo and 2.3.x in awspring. This PR too.

[WtfJoke]

I think I experience the same issue with both (secretsmanager and the parameter store).

In my case the least specific one always wins (eg. /config/application over /config/application_profile and both over /config/service)

Do you need a sample project? I might can extend the sample project. Or can I help in some other way to solve this issue? Should I test the PR?

EDIT: Woops, I thought I was in the new repo, there is an issue already open --> awspring/spring-cloud-aws#169

[peteguyatt]

@WtfJoke You're correct the same issue exists with secrets manager too @maciejwalkowiak & @MatejNedic any chance of getting this merged in?

[peteguyatt]

I'm happy to create a separate PR for the secrets manager fix too if there is a appetite for this

[WtfJoke]

@peteguyatt the PR awspring/spring-cloud-aws#195 contains fixes for both (secrets manager and parameter store).

Its probably better if we discuss in the new (migrated) issue 👍

[peteguyatt]

@WtfJoke Thanks for porting the fix to the new repo and fixing secrets manager while you were at it, would you mind adding me as a contributor to the unit tests/PR in general?