mirror of
https://github.com/freedombox/FreedomBox.git
synced 2026-04-22 10:01:45 +00:00
07e2c0ce14
- To check whether a service is running does not require root privileges. This can directly be done from a module without any action. - Since actions are allowed to be run using sudo, introducing unnecessary sub-commands increases attack surface. - Simple functions calls are unnecessarily being converted to command line invocations and involve parsing response. - There is a lot of repeated code because of this that can be eliminated. - To generalize this, we need to make all non-root system operations directly from module instead of delegating to action commands.
134 lines
4.1 KiB
Python
Executable File
134 lines
4.1 KiB
Python
Executable File
#!/usr/bin/python3
|
|
# -*- mode: python -*-
|
|
#
|
|
# This file is part of Plinth.
|
|
#
|
|
# This program is free software: you can redistribute it and/or modify
|
|
# it under the terms of the GNU Affero General Public License as
|
|
# published by the Free Software Foundation, either version 3 of the
|
|
# License, or (at your option) any later version.
|
|
#
|
|
# This program is distributed in the hope that it will be useful,
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU Affero General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU Affero General Public License
|
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
#
|
|
|
|
"""
|
|
Configuration helper for Transmission daemon.
|
|
"""
|
|
|
|
import argparse
|
|
import json
|
|
import subprocess
|
|
|
|
from plinth import action_utils
|
|
|
|
|
|
SERVICE_CONFIG = '/etc/default/transmission-daemon'
|
|
TRANSMISSION_CONFIG = '/etc/transmission-daemon/settings.json'
|
|
|
|
|
|
def parse_arguments():
|
|
"""Return parsed command line arguments as dictionary."""
|
|
parser = argparse.ArgumentParser()
|
|
subparsers = parser.add_subparsers(dest='subcommand', help='Sub command')
|
|
|
|
# Get whether service is enabled
|
|
subparsers.add_parser('get-enabled',
|
|
help='Get whether Transmission service is enabled')
|
|
|
|
# Enable service
|
|
subparsers.add_parser('enable', help='Enable Transmission service')
|
|
|
|
# Disable service
|
|
subparsers.add_parser('disable', help='Disable Transmission service')
|
|
|
|
# Merge given JSON configration with existing
|
|
merge_configuration = subparsers.add_parser(
|
|
'merge-configuration',
|
|
help='Merge given JSON configration with existing')
|
|
merge_configuration.add_argument(
|
|
'configuration',
|
|
help='JSON encoded configuration to merge')
|
|
|
|
return parser.parse_args()
|
|
|
|
|
|
def subcommand_get_enabled(_):
|
|
"""Get whether Transmission service is enabled."""
|
|
try:
|
|
with open(SERVICE_CONFIG, 'r') as file_handle:
|
|
for line in file_handle:
|
|
if line.startswith('ENABLE_DAEMON'):
|
|
value = line.split('=')[1].strip()
|
|
print('yes' if int(value) else 'no')
|
|
return
|
|
except IOError:
|
|
pass
|
|
|
|
print('no')
|
|
|
|
|
|
def subcommand_enable(_):
|
|
"""Start Transmission service."""
|
|
set_service_enable(enable=True)
|
|
action_utils.service_start('transmission-daemon')
|
|
subprocess.call(['a2enconf', 'transmission-plinth'])
|
|
action_utils.service_reload('apache2')
|
|
|
|
|
|
def subcommand_disable(_):
|
|
"""Stop Transmission service."""
|
|
subprocess.call(['a2disconf', 'transmission-plinth'])
|
|
action_utils.service_reload('apache2')
|
|
action_utils.service_stop('transmission-daemon')
|
|
set_service_enable(enable=False)
|
|
|
|
|
|
def set_service_enable(enable):
|
|
"""Enable/disable Transmission daemon; enable: boolean."""
|
|
newline = 'ENABLE_DAEMON=1\n' if enable else 'ENABLE_DAEMON=0\n'
|
|
|
|
with open(SERVICE_CONFIG, 'r') as file_handle:
|
|
lines = file_handle.readlines()
|
|
for index, line in enumerate(lines):
|
|
if line.startswith('ENABLE_DAEMON'):
|
|
lines[index] = newline
|
|
break
|
|
|
|
with open(SERVICE_CONFIG, 'w') as file_handle:
|
|
file_handle.writelines(lines)
|
|
|
|
|
|
def subcommand_merge_configuration(arguments):
|
|
"""Merge given JSON configuration with existing configuration."""
|
|
configuration = arguments.configuration
|
|
configuration = json.loads(configuration)
|
|
|
|
current_configuration = open(TRANSMISSION_CONFIG, 'r').read()
|
|
current_configuration = json.loads(current_configuration)
|
|
|
|
new_configuration = current_configuration
|
|
new_configuration.update(configuration)
|
|
new_configuration = json.dumps(new_configuration, indent=4, sort_keys=True)
|
|
|
|
open(TRANSMISSION_CONFIG, 'w').write(new_configuration)
|
|
action_utils.service_reload('transmission-daemon')
|
|
|
|
|
|
def main():
|
|
"""Parse arguments and perform all duties."""
|
|
arguments = parse_arguments()
|
|
|
|
subcommand = arguments.subcommand.replace('-', '_')
|
|
subcommand_method = globals()['subcommand_' + subcommand]
|
|
subcommand_method(arguments)
|
|
|
|
|
|
if __name__ == '__main__':
|
|
main()
|