Skip to content
Commits on Source (2)
  • nimrod's avatar
    Update the address of the flake8 pre-commit hook. · 0d5acb2b
    nimrod authored
    0d5acb2b
  • nimrod's avatar
    Improve security by restricting the usage of the response. · 00b764d8
    nimrod authored
    The service replies with the message in the request. This can be used as
    an attack vector as the reply is determined by the request and is coming
    from a shore.co.il domain. So the following precautions are taken:
    
    - Limit the request length to limit the usefulness of the response.
    - Set the response MIME type to plain text and set the
      `X-Content-Type-Options` header to `nosniff` so the browser won't
      guess the content type.
    - Set the `X-Frame-Options` header to `DENY` so it won't be used as an
      iframe.
    - Set CORS headers.
    00b764d8
......@@ -96,7 +96,7 @@ repos:
- bandit
- pyroma
- repo: https://gitlab.com/pycqa/flake8.git
- repo: https://github.com/pycqa/flake8.git
rev: 3.9.2
hooks:
- id: flake8
......
......@@ -5,25 +5,38 @@ import json
import os
import docker
from flask import Flask, request
from flask import Flask, Response, request
app = Flask(__name__)
app.config["MAX_CONTENT_LENGTH"] = 128
client = docker.from_env()
NAME = os.getenv("NC_NAME", "Nimrod Adar")
ALLOWED_ORIGIN = os.getent("ALLOWED_ORIGIN", "https://www.shore.co.il")
CONTAINER_NAME = os.getenv("NC_CONTAINER", "nextcloud-nextcloud-1")
NAME = os.getenv("NC_NAME", "Nimrod Adar")
HEADERS = {
"Access-Control-Allow-Credentials": False,
"Access-Control-Allow-Methods": "GET,POST",
"Access-Control-Allow-Origin": ALLOWED_ORIGIN,
"Cache-Control": "no-cache, no-store, max-age=0",
"X-Content-Type-Options": "nosniff",
"X-Frame-Options": "DENY",
}
@app.route("/ping")
def ping():
"""Healthcheck."""
return "pong"
return Response("pong", mimetype="text/plain")
@app.route("/send", methods=["GET", "POST"])
def send_message(): # noqa: MC0001
"""Send a notification."""
if request.method == "OPTIONS": # A CORS pre-flight request.
return Response(headers=HEADERS)
if request.method == "POST":
# Needs to be called before accessing other request parameters,
# otherwise it will be empty.
......@@ -69,4 +82,4 @@ def send_message(): # noqa: MC0001
if result.exit_code != 0:
raise RuntimeError(result.output.decode())
return message
return Response(message, mimetype="text/plain", headers=HEADERS)