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

Add sensitive for rundeck-config.groovy #493

Closed
wants to merge 2 commits into from
Closed

Add sensitive for rundeck-config.groovy #493

wants to merge 2 commits into from

Conversation

phaedriel
Copy link
Contributor

Pull Request (PR) description

  • This PR fixes that credentials would be shown in puppet log if they change
  • This PR contains some minor corrections
    • [warn]: The @param tag for parameter 'properties_dir' has no matching parameter at manifests/init.pp:224.
    • [warn]: The @param tag for parameter 'rdeck_base' has no matching parameter at manifests/init.pp:224.
    • [warn]: The @param tag for parameter 'projects_dir' has no matching parameter at manifests/config/resource_source.pp:80.
    • [warn]: Missing documentation for Puppet type alias 'Rundeck::Loglevel' at types/loglevel.pp:1.
    • [warn]: Missing documentation for Puppet type alias 'Rundeck::Sourcetype' at types/sourcetype.pp:1.

This Pull Request (PR) fixes the following issues

@Joris29
Copy link
Contributor

Joris29 commented Oct 20, 2023

Any update on this because it's still leaking changes to puppet logs? @phaedriel

@phaedriel phaedriel closed this Oct 24, 2023
@phaedriel
Copy link
Contributor Author

Any update on this because it's still leaking changes to puppet logs? @phaedriel

I closed the PR. The tests no longer pass

@phaedriel phaedriel deleted the sensitive-config branch May 4, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add sensitive option for database_config
3 participants