sdr-enthusiasts / gitbook-adsb-guide

Guide to aircraft tracking using ADS-B reception with SDR & docker containers. Published on GitBook.
https://sdr-enthusiasts.gitbook.io/ads-b/
Other
129 stars 54 forks source link

Add missing trailing '/' to /graphs1090 #111

Closed marcelstoer closed 1 year ago

marcelstoer commented 1 year ago

If you omit the trailing '/' the Nginx proxy will redirect to port 80 instead of the port in the original request URI:

❯ http -v black-pearl.local:8080/graphs1090
GET /graphs1090 HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
Host: black-pearl.local:8080
User-Agent: HTTPie/3.2.2

HTTP/1.1 301 Moved Permanently
Cache-Control: public, max-age=0
Connection: keep-alive
Content-Length: 162
Content-Type: text/html
Date: Mon, 12 Jun 2023 14:04:30 GMT
Location: http://black-pearl.local/graphs1090/
Server: nginx

<html>
<head><title>301 Moved Permanently</title></head>
<body>
<center><h1>301 Moved Permanently</h1></center>
<hr><center>nginx</center>
</body>
</html>

-> note how it redirects to "black-pearl.local/" instead of "black-pearl.local:8080/"

marcelstoer commented 1 year ago

Furthermore, we should add - MLAT_USER=${FEEDER_NAME} to the documented compose template (you have it in your example). Shall I do that in this PR or create a new one?