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

(MODULES-7141) Support Sensitive data type #49

Merged

Conversation

jpogran
Copy link
Contributor

@jpogran jpogran commented May 31, 2018

This is a port of the code in puppetlabs-dsc PR#324 to add Sensitive types to the dsc_lite module.

@Iristyle
Copy link
Contributor

@jpogran
Copy link
Contributor Author

jpogran commented Jun 1, 2018

The appveyor failure is due to testing on Puppet v4.2.3. The puppetlabs-dsc and puppetlabs-dsc_lite module differ in what Puppet Gem versions they test. The dsc_lite module still tests v4.2.3, while the dsc module removed it in commit e1205ef

@jpogran
Copy link
Contributor Author

jpogran commented Jun 1, 2018

Pending #50 merge to fix appveyor failure

@Iristyle
Copy link
Contributor

Iristyle commented Jun 1, 2018

#50 merged

@jpogran jpogran force-pushed the MODULES-7141-support-sensitive-type branch 2 times, most recently from 9cec679 to 8bd73cb Compare June 1, 2018 15:32
This commit ports code added in puppetlabs-toy-chest/puppetlabs-dsc#324

Sensitive was initially added in Puppet 4.6.0 and fully fleshed out
by Puppet 4.8.0. This module currently supports a minimum of
Puppet 4.7.0 and therefore users should be expected to use
Sensitive for password fields (example updated in README).

Added new tests for type helpers demonstrating the ability for
MSFT_Credential to allow Sensitive.
@jpogran jpogran force-pushed the MODULES-7141-support-sensitive-type branch from 8bd73cb to f292f20 Compare June 1, 2018 15:33
@jpogran jpogran changed the title (WIP)(MODULES-7141) Support Sensitive data type (MODULES-7141) Support Sensitive data type Jun 1, 2018
@RandomNoun7 RandomNoun7 merged commit 26c31b4 into puppetlabs:master Jun 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants