Simple CMS as Anwiki replacement

sitescripts/web/bin/test_server.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2013 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
@@ skipping 13 matching lines @@
 app = flask.Flask("sitescripts.web.bin.test_server")
 source = None
 
 def get_data(path):
   if source.has_static(path):
     return source.read_static(path)
 
   path = path.rstrip("/")
   if path == "":
     path = source.read_config().get("general", "defaultlocale")
-  if not "/" in path:

[Sebastian Noack, 2013/10/29 11:04:17]

In Python there is a "not in" operator, that should be used instead of not + in
operator:

if "/" not in path:

+  if "/" not in path:
     path = "%s/%s" % (path, source.read_config().get("general", "defaultpage"))
 
   locale, page = path.split("/", 1)
   for format in converters.iterkeys():
     if source.has_page(page, format):
       return process_page(source, locale, page, format).encode("utf-8")
   if source.has_localizable_file(locale, page):
     return source.read_localizable_file(locale, page)
 
   return None
 
 @app.route("/", methods = ["GET"])
 @app.route("/<path:path>", methods = ["GET"])
 def show(path=""):
   data = get_data(path)
   if data == None:
     flask.abort(404)
 
   root, ext = os.path.splitext(path)
   if ext == ".js":
-    mime = "text/js; charset=utf-8"

[Sebastian Noack, 2013/10/29 11:04:17]

This isn't the mime type for javascript, "application/javascript" would be the
correct mime type.

[Wladimir Palant, 2013/11/04 09:49:21]

Actually, application/x-javascript.

[Sebastian Noack, 2013/11/04 13:28:18]

See http://tools.ietf.org/html/rfc4329#section-7.2

[Wladimir Palant, 2013/11/04 16:11:35]

Interesting, wasn't aware of it. Apparently, nginx also changed their default a
few months ago (http://hg.nginx.org/nginx/diff/ae3fd1ca62e0/conf/mime.types), we
should make sure this is reflected in our server config.

+    mime = "application/javascript; charset=utf-8"
   elif ext == ".css":
     mime = "text/css; charset=utf-8"
   elif ext == ".png":
     mime = "image/png"
   elif ext == "":
     mime = "text/html; charset=utf-8"
   else:
     mime = "application/octet-stream"
   return data, 200, {"Content-Type": mime}
 
 if __name__ == "__main__":
   setupStderr()
   if len(sys.argv) < 2:
     print >>sys.stderr, "Usage: %s source_dir" % sys.argv[0]
     sys.exit(1)
 
   source = FileSource(sys.argv[1])
 
   # Make sure to "fix" argv to ensure that restart can succeed
   sys.argv[0:1] = ["-m", "sitescripts.web.bin.test_server"]
 
   app.run(debug=True)