[Kimchi-devel] [PATCH] [Wok 1/4] Add support to relative paths.
pvital at linux.vnet.ibm.com
pvital at linux.vnet.ibm.com
Thu Oct 6 19:09:38 UTC 2016
From: Paulo Vital <pvital at linux.vnet.ibm.com>
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 at linux.vnet.ibm.com>
---
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..f92cbc9 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 ${relative_path}/ {
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}${relative_path}/;
}
- location /websockify {
+ location ${relative_path}/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..fe62eb9 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
+# Relative Path. 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:
+#relative_path=/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..2ac9d3c 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", "relative_path", "")
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..a80289f 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'),
+ 'relative_path': config.get('server', 'relative_path'),
'version': get_version()}
diff --git a/src/wok/proxy.py b/src/wok/proxy.py
index 4b06fc2..77817a2 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.relative_path}
# 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,
+ relative_path=options.relative_path)
# 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..1d848be 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.relative_path, self.configObj)
self._load_plugins(options)
cherrypy.lib.sessions.init()
diff --git a/src/wok/utils.py b/src/wok/utils.py
index 1cdc025..409c1b6 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", "relative_path")
+ 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..82e8db8 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")
+ relative_path = config.config.get("server", "relative_path")
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('--relative_path', type="string", default=relative_path,
+ help="Relative path to server")
(options, args) = parser.parse_args()
# Update config.config with the command line values
--
2.7.4
More information about the Kimchi-devel
mailing list