[openstreetmap/openstreetmap-website] Add PWA geo protocol handler (PR #5736)

Marwin Hochfelsner notifications at github.com
Tue Mar 4 19:06:59 UTC 2025


@hlfan commented on this pull request.



> @@ -80,6 +80,11 @@ OSM = {
 
     const hash = OSM.parseHash(location.hash);
 
+    const geoURI = Object.fromEntries(new URLSearchParams(
+      params.geouri?.replaceAll(/[?;]/g, "&")?.replace(":", "=")

> Are you going to look at anything past the first semicolon?

In RFC 5870 uncp comes after crsp, so to parse `geo:57.6247,-3.6845;crs=wgs84;u=100`, I need to.
> like `:`, `&` is possible inside geouri parameters

Well then `new URL()` isn't going to help me either.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5736#discussion_r1980069152
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5736/review/2658698189 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250304/2d561e84/attachment.htm>


More information about the rails-dev mailing list