Changeset

6199:fe8222112cf4

mod_conversejs: Serve base app at / This makes things slightly less awkward for the browser to figure out which URLs belong to a PWA. The app's "start URL" was previously without the '/' and therefore was not considered within the scope of the PWA. Now the canonical app URL will always have a '/'. Prosody/mod_http should take care of redirecting existing links without the trailing / to the new URL. If you have an installation at https://prosody/conversejs then it is now at https://prosody/conversejs/ (the first URL will now redirect to the second URL if you use it). The alternative would be to make the PWA scope include the parent, i.e. the whole of https://prosody/ in this case. This might get messy if other PWAs are provided by the same site or Prosody installation, however.
author Matthew Wild <mwild1@gmail.com>
date Tue, 11 Feb 2025 13:18:38 +0000
parents 6198:7205e0348bbe
children 6200:0781470db737
files mod_conversejs/mod_conversejs.lua
diffstat 1 files changed, 2 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/mod_conversejs/mod_conversejs.lua	Sun Feb 09 17:17:10 2025 +0100
+++ b/mod_conversejs/mod_conversejs.lua	Tue Feb 11 13:18:38 2025 +0000
@@ -126,7 +126,7 @@
 module:provides("http", {
 	title = "Converse.js";
 	route = {
-		GET = function (event)
+		["GET /"] = function (event)
 			local converse_options = get_converse_options();
 
 			event.response.headers.content_type = "text/html";
@@ -175,7 +175,7 @@
 						sizes = "512x512",
 					},
 				}),
-				start_url = module:http_url(),
+				start_url = module:http_url().."/",
 				background_color = pwa_color,
 				display = "standalone",
 				scope = module:http_url().."/",