Support for loading config object from "<(/usr/bin/envsubst < template)"

[LucidOne]

configobj has an issue being passed a file descriptor (/dev/fd) via process substitution.

os.path.isfile("/dev/fd/{{ anything }}") returns False for file descriptors while os.path.exists() returns True.

test.py

#!/usr/bin/python
import sys
import configobj

print configobj.ConfigObj(sys.argv[1])

I'll update the PR with a test when I get a chance

$ cat foo.ini 
shell = /bin/bash
$ cat bar.ini.template 
shell = $SHELL
$ ./test.py foo.ini 
{'shell': '/bin/bash'}
$ ./test.py <(envsubst < bar.ini.template )
{'shell': '/bin/bash'}

This is needed in cloud environments where you may not want to store passwords on disk in an unencrypted form or have variables passed into a container environment.

[coveralls]

Coverage decreased (-47.9%) to 33.926% when pulling ae4b6ab on LucidOne:process_substitution into 45fbf1b on DiffSK:master.

[LucidOne]

I'm not sure why coverage dropped, but it looks like the tests work.
Let me know if you have any other feedback.

[LucidOne]

Is there anything left to get this merged?

[LucidOne]

Any update on this?

[LucidOne]

Is it possible to get this merged? I'd like to improve the letsencrypt certbot docker container for doing dns based wildcard cert requests and this is blocking me.

[LucidOne]

Would it help to rebase this?

[jelmer]

Hi, what are the security implications of these changes? How trusted are files read by configobj today?