echo GuSSMParameter values by default

[akash1810]

What does this change?

The value of GuSSMParameter will be the path into parameter store. This isn't private, so setting noEcho should not be the default.

I observed this whilst applying https://github.com/guardian/deploy-tools-platform/pull/323. The value of LoggingStreamName is defaulted and with noEcho set to true, it caused for an interesting CFN update as had to triple check the redacted value.

It is still possible to set noEcho and have added a test to demonstrate this.

Does this change require changes to existing projects or CDK CLI?

No.

How to test

Tests have been added.

How can we measure success?

Better default behaviour of the GuSSMParameter.

Have we considered potential risks?

n/a

[jamie-lynch]

The GuSSMParameter is removed in #141 so this problem goes away. In that PR I updated one place where it was used to set noEcho to true explicitly which maybe can be removed?

[akash1810]

The GuSSMParameter is removed in #141 so this problem goes away. In that PR I updated one place where it was used to set noEcho to true explicitly which maybe can be removed

Ah, good point! Yeah, removing the explicit noEcho would be preferred.