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

Split the attributes to the client and server areas #150

Merged
merged 2 commits into from
Dec 23, 2016

Conversation

artem-sidorenko
Copy link
Member

Fixes GH-53

and have it the same way:
 - only crypto related parameters are passed as template var
 - all other attributes are referenced directly via node obj from template
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 6d929fd on artem-sidorenko:subnamespace into c1fbf18 on dev-sec:master.

Copy link
Member

@chris-rock chris-rock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work @artem-sidorenko

@chris-rock chris-rock merged commit 04e0374 into dev-sec:master Dec 23, 2016
@artem-sidorenko artem-sidorenko deleted the subnamespace branch December 23, 2016 11:16
artem-sidorenko added a commit to artem-forks/chef-ssh-hardening that referenced this pull request Dec 23, 2016
Due to the parallel merge of dev-secGH-151 and dev-secGH-150
artem-sidorenko added a commit to artem-forks/chef-ssh-hardening that referenced this pull request Dec 23, 2016
Due to the parallel merge of dev-secGH-151 and dev-secGH-150
artem-sidorenko added a commit to artem-forks/chef-ssh-hardening that referenced this pull request Dec 23, 2016
in order to provide a better overview on defaults
artem-sidorenko added a commit to artem-forks/chef-ssh-hardening that referenced this pull request Dec 23, 2016
in order to provide a better overview on defaults
@schurzi schurzi changed the title Split the attribues to the client and server areas Split the attributes to the client and server areas May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Harmonize API
3 participants