-
Couldn't load subscription status.
- Fork 263
Major refactoring of docker containers/frontend #381
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
base: gh-pages
Are you sure you want to change the base?
Changes from all commits
443b91d
540a4ef
ce87c39
ef088ba
1668176
76ac381
11633c5
b6b10e8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,54 @@ | ||
| { | ||
| "zoom": 13, | ||
| "center": [ | ||
| 50.8503, | ||
| 4.3517 | ||
| ], | ||
| "language": "en", | ||
| "layers": [ | ||
| { | ||
| "name": "openstreetmap.org", | ||
| "url": "//tile.openstreetmap.org/{z}/{x}/{y}.png", | ||
| "attribution": "© <a href=\"https://www.openstreetmap.org/copyright/en\">OpenStreetMap</a> contributors" | ||
| }, | ||
| { | ||
| "name": "openstreetmap.de", | ||
| "url": "//tile.openstreetmap.de/tiles/osmde/{z}/{x}/{y}.png", | ||
| "attribution": "<a target=\"_blank\" href=\"http://www.openstreetmap.org/\">Karte hergestellt aus OpenStreetMap-Daten</a> | Lizenz: <a rel=\"license\" target=\"_blank\" href=\"http://opendatacommons.org/licenses/odbl/\">Open Database License (ODbL)</a>" | ||
| } | ||
| ], | ||
| "overlays":[ | ||
| { | ||
| "name": "Hiking", | ||
| "url": "//tile.waymarkedtrails.org/hiking/{z}/{x}/{y}.png", | ||
| "attribution": "© <a href=\"http://waymarkedtrails.org\">Sarah Hoffmann</a> (<a href=\"https://creativecommons.org/licenses/by-sa/3.0/\">CC-BY-SA</a>)" | ||
| }, | ||
| { | ||
| "name": "Bike", | ||
| "url": "//tile.waymarkedtrails.org/cycling/{z}/{x}/{y}.png", | ||
| "attribution": "© <a href=\"http://waymarkedtrails.org\">Sarah Hoffmann</a> (<a href=\"https://creativecommons.org/licenses/by-sa/3.0/\">CC-BY-SA</a>)" | ||
| }, | ||
| { | ||
| "name": "Small Components", | ||
| "url": "https://tools.geofabrik.de/osmi/tiles/routing/{z}/{x}/{y}.png" | ||
| } | ||
|
|
||
| ], | ||
| "services": [ | ||
| { | ||
| "label": "Car", | ||
| "path": "https://routing.openstreetmap.de/routed-car/route/v1", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The former default was There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sounds good. My thought was that someone running the docker for the first time would like to have something working immediately before changing the config, to simply try it out. Hence defaulting to something that works. But I'll change it back to localhost There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So would you instead only have one service, pointing to localhost:5000? Or one pointing to localhost and the other two to the routing.openstreetmap.de for foot and bike? |
||
| "debug": "car" | ||
| }, | ||
| { | ||
| "label": "Bike", | ||
| "path": "https://routing.openstreetmap.de/routed-bike/route/v1", | ||
| "debug": "bike" | ||
| }, | ||
| { | ||
| "label": "Foot", | ||
| "path": "https://routing.openstreetmap.de/routed-foot/route/v1", | ||
| "debug": "foot" | ||
| } | ||
| ] | ||
| } | ||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this note? With the current docker image (
sha256:d05515fff579039bc3b45289003174b66d00a72c9a04d841656146cce6aa24ea) the debug map works just fine out of the boxThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll be honest, it's been a while since I've first made this so I'm not too sure. Looking at debug/index.html, it looks like there is a hardcoded mapbox access token, which I think is expired? I just tried running the default docker image on my VPS, and I just get a black page, with this error message on the console:
Not sure what is supposed to be running on port 5000 (osrm frontend runs on port 9966 by default), so I guess that's what I meant with my edit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Mapbox token is used for Mapbox layers in the normal (non-debug) view. The osrm-backend instance runs on port 5000 by default. I don't know what you are doing, but if you run the OSRM backend and frontend Docker images as they are, the debug view works just fine. You may want to specify the correct backend by running the frontend with the command

-e OSRM_BACKEND="http://whatever.yourdomain:Port", as mentioned in the README.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, I can then remove that extra line in the README. Other thoughts on this PR?