FR: Also expose placeholders for geo ip information

[melink14]

I was looking at caddy geoip capabilities so I could augment my access logs with location information (for later processing and monitoring) and it seems like this plugin is the only geolocation plugin that was ported to caddy v2.

caddy v1 had https://github.com/aablinov/caddy-geoip which exposed placeholders and I think this plugin is best situated to do something similar.

There might be an issue of redundancy since once you define placeholders then one could use either the custom matchers already defined by the plugin or just use expression to create any condition they want using the value of the placeholders.

(On the other hand, it would make extensions like #3 to implement since no expression syntax would need to change)

It seems like using a handler to add placeholders is pretty easy but it is a bit of a philosophical change vs having a matcher. My own opinion is providing placeholders is more elegant software solution but then we'd have to worry about migration.

Let me know what you think!

[ermiaqasemi]

Hey, I believe this would be very useful feature, is there any plan to add this placeholders ?

[sandstrom]

Here is another addon that does it.

But would be great if that functionality was available in this addon too (so we didn't need two).

https://github.com/shift72/caddy-geo-ip

[simono41]

I would also like it.

At the moment it is sufficient to set the log level to DEBUG and then access it with Promtail. So that I have statistics for my Grafana dashboard.

    log {
            output file /logs/caddy_main.log {
                    roll_size 100MiB
                    roll_keep 5
                    roll_keep_for 100d
            }
            format json
            level DEBUG
    }

[ale-rinaldi]

Hello,

I see it would be really useful and I faced myself the usefulness of using expressions to deal with complex scenarios without headaches.

That's on my list, but I don't know when I'll be able to complete this. As always, PRs are very welcome :)