1

I am working on a tool (Python, may or may not be important) that will allow a user to maintain a configuration file containing arbitrary shell and/or language code to be executed on particular events and intervals. The code will potentially involve multiple statements.

I understand there are potential behavioral and security risks associated with executing arbitrary code from a file, but I don't think this is a concern because the configuration file is managed only by the end user. However, let me know if I'm overlooking something major.

My main question: what is the best way to store this code in a configurable way?


Some concepts I'm considering to store the code...

In a shared config file (which many Python libraries already look for):

# setup.cfg
[my_program]
command_1 = cd /tmp && touch foobar

In a YAML file:

# my_program.yaml
command_1: cd /tmp && touch foobar

In a source file as arguments to subprocess.call():

# settings.py
command_1 = [['cd', '/tmp'], ['touch', 'foobar']] # requires special handling of 'cd'

In a source file as a function:

# settings.py
import os
import subprocess
def command_1():
    os.chdir('/tmp')
    subprocess.call(['touch', 'foobar'])
5
  • IMHO that is somewhat a matter of taste, a matter how complex your config code may be (for example, may it contain multiple functions calling each other?) and a matter of the kind of end-user you have in mind.
    – Doc Brown
    Commented Jul 12, 2013 at 5:54
  • @DocBrown: the end users are programmers setting up build and test commands for various code repositories. Commented Jul 12, 2013 at 11:45
  • A note, your last example doesn't do the same thing as the first two. The third can go either way depending on implementation, but removes the option from the user.
    – Izkata
    Commented Jul 14, 2013 at 23:55
  • And I wasn't even referring to the point user13739 makes.. Forgot about that
    – Izkata
    Commented Jul 15, 2013 at 0:18
  • @Izkata: It's fixed now. Commented Jul 15, 2013 at 0:59

3 Answers 3

2

How about storing the commands in an actual executable file? The end user creates a bash/python/ruby script, and all they supply to your configuration is the name of the script to execute.

This allows the script to be arbitrarily simple or complex, and also lets the end user use any scripting language supported by their own environment.

Finally, they will need permissions to make the script file executable for this to work, which means they already had permissions to create and run any script anyways.

3
  • What if I need this file to store many different commands? Should there me multiple scripts? A standard set of arguments I will pass to their scripts? They also tell me what arguments to pass to their script for each command? Commented Jul 12, 2013 at 11:48
  • 1
    @Jace Browning Yes, there should be multiple scripts. If the user wants code reuse, the can always make all of their scripts call into a common codebase. As far as arguments, I would keep it as simple as possible to start and pass a standard set of arguments. Commented Jul 12, 2013 at 13:17
  • @ChrisPitman, any examples will make better to understand the ideology you are referring. Thanks... Commented Mar 21, 2018 at 13:50
1

Well, for your example script only the first two methods will actually work. It has a cd in it, and therefore the two commands must be part of the same shell script.

Therefore, if you want to use commands that have the form of a shell script, you have to store them in form of a string anyway. If the code you want to store may become longer, YAML might be the better idea, since it can handle multi-line strings better.

0
0

One option I'm considering is to have users create a Makefile with specific target names.

Advantages:

  • all commands are locally testable

Disadvantages:

  • requires make

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.