Set TARGET_USER only if it's a non-empty string

tohuwabohu / puppet-duplicity

Puppet module to manage backups based on duplicity.

Apache License 2.0

8 stars 33 forks source link

Set TARGET_USER only if it's a non-empty string #35

Closed stefanopini closed 7 years ago

stefanopini commented 7 years ago

Avoid to set TARGET_USER and TARGET_PASS to empty strings if they are not set. Duply check if these vaules are set and in that case it will add "@TARGET_USER:TARGET_PASS" to the TARGET url, causing an error if they are empty strings.

tohuwabohu commented 7 years ago

LGTM - can you please take care of the failing rspec tests? Thanks

stefanopini commented 7 years ago

Sure! It should be ok now :-)

tohuwabohu commented 7 years ago

I've released version 4.4.2 to the forge, thanks for the contribution.