saltstack-formulas / apache-formula

Set up and configure the Apache HTTP server
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
50 stars 284 forks source link

Fix wrong yaml parsing #253

Closed javierbertoli closed 5 years ago

javierbertoli commented 5 years ago

Due to this SaltStack changes to use unicode strings, the use of sites:<vhostname>:Formula_Append to manage vhosts with apache/vhosts/standard.sls introduces parsing errors of the pillar when applying apache/config.sls

testhost.com:
    Data failed to compile:
----------
    Rendering SLS 'base:apache.config' failed: while parsing a flow mapping
  in "<unicode string>", line 32, column 798:
     ... t.com'}, 'testsite': {'AcceptPathInfo': 'On', 'Server ... 
                              ^
expected ',' or '}', but got '<scalar>'
  in "<unicode string>", line 32, column 1551:
     ... default\n    # To re-enable it\'s recommended to enable access t ... 

Applied the solution documented here and here and now it works again.

aboe76 commented 5 years ago

@javierbertoli merged it.

alxwr commented 5 years ago

@javierbertoli Thanks for this and for the explanation.