Merge pull request redhat-openstack#32 from ChrisPortman/option_ordering

Implement Consistent Option Ordering
strider · Apr 15, 2014 · dffda8a · dffda8a
2 parents 1426c30 + 02037f0
commit dffda8a
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/templates/sshd_config.erb b/templates/sshd_config.erb
@@ -1,9 +1,18 @@
 # File is managed by Puppet
+<%- options = scope.lookupvar('ssh::server::merged_options') -%>
+<%- if port = options.delete('Port') -%>
+Port <%= port %>
+<%- end -%>
+<%- if listen = options.delete('ListenAddress') -%>
+ListenAddress <%= listen %>
+<%- end -%>
 
-<%- scope.lookupvar('ssh::server::merged_options').sort_by{ |sk| (sk.to_s.downcase.include? "match") ? sk.to_s : '' }.each do |k, v| -%>
+<%- options.keys.sort_by{ |sk| (sk.to_s.downcase.include? "match") ? 'zzz' + sk.to_s : sk.to_s }.each do |k| -%>
+<%- v = options[k] -%>
 <%- if v.is_a?(Hash) -%>
 <%= k %>
-<%- v.sort.each do |key, value| -%>
+<%- v.keys.sort.each do |key| -%>
+    <%- value = v[key] -%>
     <%- if value.is_a?(Array) -%>
     <%- value.each do |a| -%>
     <%= key %> <%= a %>