Feb 13, 2025
diff --git a/templates/Caddyfile.j2 b/templates/Caddyfile.j2

 root * {{ caddy.site_dir }}

 request_body {
 max_size 25MB  # Limit from GitHub.
 }

 # https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#delivery-headers
 @valid_webhook {
 path /gh/*

 root * {{ caddy.site_dir }}

 request_body {
 max_size 0
 }

 {% for site, path in repos.items() %}
 import subproject {{ site }} {{ path | default(site, true) }}
 {% endfor %}
diff --git a/webhook/webhook.py b/webhook/webhook.py

    We only handle ping and push events (this is enforced by the Caddyfile).
    """
    # Verify some input parameters.
    if request.content_length > 25_000_000:  # Limit from GitHub.
        raise web.HTTPBadRequest(reason='Request too large')

    # This should be guarded against by Caddy, but check anyway.
    # https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#delivery-headers
    try:
    site_dir = Path(os.environ.get('SITE_DIR', 'sites')).resolve()
    assert site_dir.is_dir()

    app = web.Application()
    app = web.Application(client_max_size=25_000_000)  # Limit from GitHub.
    app['site_dir'] = site_dir
    app.add_routes([
        web.post('/gh/{repo}', github_webhook),
Original file line number	Diff line number	Diff line change
Expand Up		@@ -33,6 +33,10 @@ http://{{ caddy.addresses.webhook }} {

		root * {{ caddy.site_dir }}

		request_body {
		max_size 25MB # Limit from GitHub.
		}

		# https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#delivery-headers
		@valid_webhook {
		path /gh/*
Expand DownExpand Up		@@ -85,6 +89,10 @@ http://{{ caddy.addresses.main }}, http://{{ ansible_fqdn }} {

		root * {{ caddy.site_dir }}

		request_body {
		max_size 0
		}

		{% for site, path in repos.items() %}
		import subproject {{ site }} {{ path \| default(site, true) }}
		{% endfor %}
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -88,10 +88,6 @@ async def github_webhook(request: web.Request):

		We only handle ping and push events (this is enforced by the Caddyfile).
		"""
		# Verify some input parameters.
		if request.content_length > 25_000_000: # Limit from GitHub.
		raise web.HTTPBadRequest(reason='Request too large')

		# This should be guarded against by Caddy, but check anyway.
		# https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#delivery-headers
		try:
Expand DownExpand Up		@@ -183,7 +179,7 @@ def create_app():
		site_dir = Path(os.environ.get('SITE_DIR', 'sites')).resolve()
		assert site_dir.is_dir()

		app = web.Application()
		app = web.Application(client_max_size=25_000_000) # Limit from GitHub.
		app['site_dir'] = site_dir
		app.add_routes([
		web.post('/gh/{repo}', github_webhook),
Expand Down