Lint: Fix linter errors

Signed-off-by: Knut Ahlers <knut@ahlers.me>

Makefile

@@ -1,9 +1,7 @@
-publish:
+default:
-	curl -sSLo golang.sh https://raw.githubusercontent.com/Luzifer/github-publish/master/golang.sh
-	bash golang.sh
 
 update_example_post:
 	curl -X POST --data-binary @example/postmap.json -o example/postmap.png localhost:5000/map.png
 
 update_example_get:
-	curl -o example/map.png 'localhost:5000/map.png?center=53.5438,9.9768&zoom=15&size=800x500&markers=color:blue|53.54129165,9.98420576699353&markers=color:yellow|53.54565525,9.9680555636958'
+	curl -o example/map.png 'localhost:5000/map.png?center=53.5438,9.9768&zoom=15&size=800x500&markers=color:blue|53.54129165,9.98420576699353&markers=color:yellow|53.54565525,9.9680555636958&markers=size:tiny|color:red|53.54846472989871,9.978977621091543'

README.md

@@ -27,9 +27,10 @@ All map operations are made against the `/map.png` endpoint of the server and ar
     &size=800x500
     &markers=color:blue|53.54129165,9.98420576699353
     &markers=color:yellow|53.54565525,9.9680555636958
+    &markers=size:tiny|color:red|53.54846472989871,9.978977621091543
 ```
 
-The map center is set to a coordinate within Hamburg, Germany with a zoom level of 15. Additionally there are two markers set: One `blue` marker to the Elbphilharmonie, one `yellow` marker to the Hard Rock Cafe Hamburg. The example above (of course without the line breaks) produced this image:
+The map center is set to a coordinate within Hamburg, Germany with a zoom level of 15. Additionally there are two markers set: One `blue` marker to the Elbphilharmonie, one `yellow` marker to the Hard Rock Cafe Hamburg and a `tiny` `red` marker on the St. Michaels church. The example above (of course without the line breaks) produced this image:
 
 ![](example/map.png)
 

example/postmap.json

@@ -18,6 +18,14 @@
         "lat": 53.54565525,
         "lon": 9.9680555636958
       }
+    },
+    {
+      "color": "red",
+      "size": "tiny",
+      "coord": {
+        "lat": 53.54846472989871,
+        "lon": 9.978977621091543
+      }
     }
   ],
   "overlays": [
@@ -25,4 +33,4 @@
   ],
   "width": 800,
   "zoom": 15
-}
+}

main.go

@@ -67,7 +67,7 @@ func main() {
 	rateLimit.SetIPLookups([]string{"X-Forwarded-For", "RemoteAddr", "X-Real-IP"})
 
 	r := mux.NewRouter()
-	r.HandleFunc("/status", func(res http.ResponseWriter, r *http.Request) { http.Error(res, "I'm fine", http.StatusOK) })
+	r.HandleFunc("/status", func(res http.ResponseWriter, _ *http.Request) { http.Error(res, "I'm fine", http.StatusOK) })
 	r.Handle("/map.png", tollbooth.LimitFuncHandler(rateLimit, handleMapRequest)).Methods("GET")
 	r.Handle("/map.png", tollbooth.LimitFuncHandler(rateLimit, handlePostMapRequest)).Methods("POST")
 
@@ -77,7 +77,7 @@ func main() {
 		ReadHeaderTimeout: time.Second,
 	}
 
-	logrus.WithField("version", version).Info("staticmap started")
+	logrus.WithField("version", version).WithField("addr", cfg.Listen).Info("staticmap started")
 	if err = server.ListenAndServe(); err != nil {
 		logrus.WithError(err).Fatal("running HTTP server")
 	}
@@ -173,7 +173,7 @@ func parseCoordinate(coord string) (s2.LatLng, error) {
 	}
 
 	parts := strings.Split(coord, ",")
-	if len(parts) != 2 { //nolint:gomnd
+	if len(parts) != 2 { //nolint:mnd
 		return s2.LatLng{}, errors.New("Coordinate not in format lat,lon")
 	}
 
@@ -200,7 +200,7 @@ func parseSize(size string) (x, y int, err error) {
 	}
 
 	parts := strings.Split(size, "x")
-	if len(parts) != 2 { //nolint:gomnd
+	if len(parts) != 2 { //nolint:mnd
 		return 0, 0, errors.New("Size not in format 600x300")
 	}
 

map.go

@@ -14,7 +14,7 @@ import (
 	"github.com/pkg/errors"
 )
 
-//nolint:gomnd // these are the "constant" definitions
+//nolint:mnd // these are the "constant" definitions
 var markerColors = map[string]color.Color{
 	"black":  color.RGBA{R: 145, G: 145, B: 145, A: 0xff},
 	"brown":  color.RGBA{R: 178, G: 154, B: 123, A: 0xff},
@@ -30,7 +30,7 @@ var markerColors = map[string]color.Color{
 
 type markerSize float64
 
-//nolint:gomnd // these are the "constant" definitions
+//nolint:mnd // these are the "constant" definitions
 var markerSizes = map[string]markerSize{
 	"tiny":  10,
 	"mid":   15,
@@ -97,7 +97,7 @@ func generateMap(opts generateMapConfig) (io.Reader, error) {
 
 	if opts.Markers != nil {
 		for _, m := range opts.Markers {
-			ctx.AddMarker(staticMap.NewMarker(m.pos, m.color, float64(m.size)))
+			ctx.AddObject(staticMap.NewMarker(m.pos, m.color, float64(m.size)))
 		}
 	}
 

postmap.go

@@ -105,7 +105,7 @@ func (p postMapOverlay) toOverlays() ([]*staticMap.TileProvider, error) {
 
 		result = append(result, &staticMap.TileProvider{
 			Name:       fmt.Sprintf("%x", sha256.Sum256([]byte(pat))),
-			TileSize:   256, //nolint:gomnd
+			TileSize:   256, //nolint:mnd
 			URLPattern: pat,
 		})
 	}