[Kimchi-devel] [PATCH] [Wok 1/4] Add support to relative paths.

From: Paulo Vital <pvital(a)linux.vnet.ibm.com&gt; This patch adds support to handle the use of relative paths in configuration and command line, as well, set the backend to use the new path in URL's. This is part of solution to: https://github.com/kimchi-project/kimchi/issues/733 and https://github.com/kimchi-project/wok/issues/25 Signed-off-by: Paulo Vital <pvital(a)linux.vnet.ibm.com&gt; --- src/nginx/wok.conf.in | 6 +++--- src/wok.conf.in | 5 +++++ src/wok/config.py.in | 1 + src/wok/model/config.py | 1 + src/wok/proxy.py | 8 +++++--- src/wok/reqlogger.py | 2 +- src/wok/server.py | 2 +- src/wok/utils.py | 11 ++++++++++- src/wokd.in | 3 +++ 9 files changed, 30 insertions(+), 9 deletions(-) diff --git a/src/nginx/wok.conf.in b/src/nginx/wok.conf.in index 512b00b..5d2bb17 100644 --- a/src/nginx/wok.conf.in +++ b/src/nginx/wok.conf.in @@ -56,15 +56,15 @@ server { add_header X-Content-Type-Options nosniff; add_header X-XSS-Protection "1; mode=block"; - location / { + location ${server_root}/ { proxy_pass http://127.0.0.1:${cherrypy_port}; proxy_set_header Host $host; proxy_set_header X-Real-IP $remote_addr; proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for; - proxy_redirect http://127.0.0.1:${cherrypy_port}/ https://$host:${proxy_ssl_port}/; + proxy_redirect http://127.0.0.1:${cherrypy_port}/ https://$host:${proxy_ssl_port}${server_root}/; } - location /websockify { + location ${server_root}/websockify { proxy_pass http://websocket; proxy_http_version 1.1; proxy_set_header Upgrade $http_upgrade; diff --git a/src/wok.conf.in b/src/wok.conf.in index 67dc75a..5e04b20 100644 --- a/src/wok.conf.in +++ b/src/wok.conf.in @@ -43,6 +43,11 @@ # Max request body size in KB, default value is 4GB #max_body_size = 4 * 1024 * 1024 +# Wok server root. Set the following variable to configure any relative path to +# the server. For example, to have Wok pointing to https://localhost:8001/wok/ +# uncomment the following: +#server_root=/wok + [logging] # Log directory #log_dir = @localstatedir@/log/wok diff --git a/src/wok/config.py.in b/src/wok/config.py.in index cbe585c..46544fa 100644 --- a/src/wok/config.py.in +++ b/src/wok/config.py.in @@ -272,6 +272,7 @@ def _get_config(): config.set("server", "ssl_key", "") config.set("server", "environment", "production") config.set('server', 'max_body_size', '4*1024*1024') + config.set("server", "server_root", "") config.add_section("authentication") config.set("authentication", "method", "pam") config.set("authentication", "ldap_server", "") diff --git a/src/wok/model/config.py b/src/wok/model/config.py index 415098a..3748a6e 100644 --- a/src/wok/model/config.py +++ b/src/wok/model/config.py @@ -28,4 +28,5 @@ class ConfigModel(object): return {'ssl_port': config.get('server', 'ssl_port'), 'websockets_port': config.get('server', 'websockets_port'), 'auth': config.get('authentication', 'method'), + 'server_root': config.get('server', 'server_root'), 'version': get_version()} diff --git a/src/wok/proxy.py b/src/wok/proxy.py index 4b06fc2..5f646e4 100644 --- a/src/wok/proxy.py +++ b/src/wok/proxy.py @@ -37,7 +37,7 @@ from wok.utils import run_command HTTP_CONFIG = """ server { listen %(host_addr)s:%(proxy_port)s; - rewrite ^/(.*)$ https://$host:%(proxy_ssl_port)s/$1 redirect; + rewrite ^/(.*)$ https://$host:%(proxy_ssl_port)s%(rel_path)s/$1 redirect; } """ @@ -88,7 +88,8 @@ def _create_proxy_config(options): if options.https_only == 'false': http_config = HTTP_CONFIG % {'host_addr': options.host, 'proxy_port': options.port, - 'proxy_ssl_port': options.ssl_port} + 'proxy_ssl_port': options.ssl_port, + 'rel_path': options.server_root} # Read template file and create a new config file # with the specified parameters. @@ -104,7 +105,8 @@ def _create_proxy_config(options): cert_pem=cert, cert_key=key, max_body_size=eval(options.max_body_size), session_timeout=options.session_timeout, - dhparams_pem=dhparams_pem) + dhparams_pem=dhparams_pem, + server_root=options.server_root) # Write file to be used for nginx. config_file = open(os.path.join(nginx_config_dir, "wok.conf"), "w") diff --git a/src/wok/reqlogger.py b/src/wok/reqlogger.py index 37190dc..b51abf1 100644 --- a/src/wok/reqlogger.py +++ b/src/wok/reqlogger.py @@ -40,7 +40,7 @@ from wok.utils import remove_old_files # Log search setup FILTER_FIELDS = ['app', 'date', 'ip', 'req', 'status', 'user', 'time'] -LOG_DOWNLOAD_URI = "/data/logs/%s" +LOG_DOWNLOAD_URI = "data/logs/%s" LOG_DOWNLOAD_TIMEOUT = 6 LOG_FORMAT = "[%(date)s %(time)s %(zone)s] %(req)-6s %(status)s %(app)-11s " \ "%(ip)-15s %(user)s: %(message)s\n" diff --git a/src/wok/server.py b/src/wok/server.py index 04f4330..fecca48 100644 --- a/src/wok/server.py +++ b/src/wok/server.py @@ -178,7 +178,7 @@ class Server(object): cfg[ident] = {'tools.wokauth.on': True} self.app = cherrypy.tree.mount(WokRoot(model_instance, dev_env), - config=self.configObj) + options.server_root, self.configObj) self._load_plugins(options) cherrypy.lib.sessions.init() diff --git a/src/wok/utils.py b/src/wok/utils.py index 1cdc025..0b74fd8 100644 --- a/src/wok/utils.py +++ b/src/wok/utils.py @@ -39,7 +39,7 @@ from datetime import datetime, timedelta from multiprocessing import Process, Queue from threading import Timer -from wok.config import paths, PluginPaths +from wok.config import config, paths, PluginPaths from wok.exception import InvalidParameter, TimeoutExpired from wok.stringutils import decode_value @@ -71,6 +71,14 @@ def _load_plugin_conf(name): (plugin_conf, e.message)) +def _check_plugin_relative_path(plugin_config): + rel_path = config.get("server", "server_root") + plugin_uri = plugin_config['wok']['uri'] + if (rel_path is not "") and (not plugin_uri.startswith(rel_path)): + plugin_config['wok']['uri'] = rel_path + plugin_uri + return plugin_config + + def get_enabled_plugins(): plugin_dir = paths.plugins_dir try: @@ -82,6 +90,7 @@ def get_enabled_plugins(): plugin_config = _load_plugin_conf(name) try: if plugin_config['wok']['enable']: + plugin_config = _check_plugin_relative_path(plugin_config) yield (name, plugin_config) except (TypeError, KeyError): continue diff --git a/src/wokd.in b/src/wokd.in index 962581d..c1b302c 100644 --- a/src/wokd.in +++ b/src/wokd.in @@ -51,6 +51,7 @@ def main(options): websockets_port = config.config.get("server", "websockets_port") session_timeout = config.config.get("server", "session_timeout") runningEnv = config.config.get("server", "environment") + server_root = config.config.get("server", "server_root") logDir = config.config.get("logging", "log_dir") logLevel = config.config.get("logging", "log_level") @@ -85,6 +86,8 @@ def main(options): help="Running environment of wok server") parser.add_option('--test', action='store_true', help="Run server in mock model") + parser.add_option('--server_root', type="string", default=server_root, + help="Relative path to server") (options, args) = parser.parse_args() # Update config.config with the command line values -- 2.7.4