Skip to content

Conversation

@Laogeodritt
Copy link
Contributor

In generated pg_hba.conf, fix missing spaces between arguments if an argument is longer than the column width allotted to that argument in the line.

An example pillar excerpt:

postgres:
    # ...
    acls:
        - ['local', 'database1', 'samehost', 'user']
        - ['local', 'database1,database2,database3', 'user', 'samehost', 'md5']

Before patch, outputs are:

local    database1       user            samehost                md5
local   database1,database2,database3user            samehost                md5

After patch:

local    database1       user            samehost                md5
local   database1,database2,database3 user            samehost                md5

Second line is less readable since the columns aren't strictly adhered to anymore, but it is correctly read by postgresql at least.

In generated pg_hba.conf, fix missing spaces between arguments if an argument is longer than the column width allotted to that argument in the line.
@javierbertoli
Copy link
Member

Great catch! Thanks!

@javierbertoli javierbertoli merged commit 62c54f0 into saltstack-formulas:master Oct 9, 2016
@Laogeodritt Laogeodritt deleted the laogeodritt-fix-acl branch October 9, 2016 18:19
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.

2 participants