Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make tiles URL handle double slashes /tiles//12/2146/1390.mvt #693

Open
felixguendling opened this issue Jan 7, 2025 · 3 comments
Open

Comments

@felixguendling
Copy link
Member

No description provided.

@Lucas-C
Copy link

Lucas-C commented Jan 14, 2025

Tested with a fresh build based on commit 860da4d
and it seems to work fine when I query http://localhost:1280/tiles//15/16597/11273.mvt locally:
the file can be downloaded correctly.

@felixguendling
Copy link
Member Author

@jbruechert can you check? I just took your comment on Matrix and made an issue out of it. The problem a few days ago was that the Nginx proxy added a slash somewhere and somehow the tiles didn't reach the frontend.

If there's no problem - even better. Thank you @Lucas-C for checking!

@jbruechert
Copy link
Contributor

I think the problem is that it does still returns a syntactically correct tile, just often without anything useful in it.

I then occasionally see things like this in the log:

Jan 17 22:41:13 crunchy motis[520788]: 2025-01-17T21:41:13Z [VERIFY FAIL] invalid zoom level 900625640

The broken nginx config is

    location /tiles {
        proxy_pass http://localhost:8080/tiles/;
    }

A slash after /tiles fixes it, preventing the double slash from being passed on:

    location /tiles/ {
        proxy_pass http://crunchy:8080/tiles/;
    }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants