search

9seconds / topographer

Self-hosted lenient geoip service
MIT License
28 stars 4 forks source link

Discussion: Missing route /{ip} #13

Closed nesc58 closed 3 years ago

nesc58 commented 3 years ago

Hi again, thanks a lot for this great project. you shall be blessed

Years ago I have created a pull request to handle the following:

The route "/{ip}" is added to lookup a single IP via GET method.

Is it possible to readd this route?

Background: From mine point of view it makes more sense to resolve a single IP due a GET Request instead of sending a POST request filled with a single IP. POST feels more like "Create a resource" GET feels more like "Find something for me"

Kind regards!

9seconds commented 3 years ago

Sure, I've missed that. Will do!

nesc58 commented 3 years ago

Works like a charm. Thank you so much