Software /
code /
prosody-modules
Changeset
4930:13070c6a7ce8
mod_http_muc_log: Fix exception on lack of trailing slash in room path
A request to /room leads to the match call returning nil which in turn
calls nodeprep(nil). In Prosody 0.11.x this does nothing and simply
returns the nil, while in 0.12 it is an error.
Now it redirects to the calendar view at /room/ - even for non-existant
rooms.
Discovered at a deployment with http_paths = { muc_log = "/" } and
requests to /robots.txt and similar, which now result in a uses redirect
before returning 404.
author | Kim Alvefur <zash@zash.se> |
---|---|
date | Fri, 22 Apr 2022 14:29:32 +0200 |
parents | 4929:65438e4ba563 |
children | 4931:f4a9e804c457 |
files | mod_http_muc_log/mod_http_muc_log.lua |
diffstat | 1 files changed, 4 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/mod_http_muc_log/mod_http_muc_log.lua Mon Apr 18 23:02:04 2022 +0200 +++ b/mod_http_muc_log/mod_http_muc_log.lua Fri Apr 22 14:29:32 2022 +0200 @@ -275,6 +275,10 @@ local request, response = event.request, event.response; local room, date = path:match("^([^/]+)/([^/]*)/?$"); + if not room then + response.headers.location = url.build({ path = path .. "/" }); + return 303; + end room = nodeprep(room); if not room then return 400;