unkey icon indicating copy to clipboard operation
unkey copied to clipboard

Display the city rather than the IP address in the audit logs

Open harshsbhat opened this issue 1 year ago • 6 comments

Preliminary Checks

  • [X] I have reviewed https://unkey.com/docs for existing features that would solve my problem

  • [X] I have searched for existing feature requests: https://github.com/unkeyed/unkey/issues

  • [X] This issue is not a question, general help request, or anything other than a feature request directly related to Unkey. Please ask questions in our Discord community: https://unkey.com/discord.

Is your feature request related to a problem? Please describe.

Currently we display the IP address in the audit logs which looks something like this:

image

Describe the solution

But as the end user, I would be more interested in the city or a location which is humanly readable rather than something like IP address

Describe alternatives you have considered (if any)

No response

Additional context

No response

harshsbhat avatar Oct 07 '24 14:10 harshsbhat

The new logs @ogzhanolguncu is working on may provide insights into that, but we'll have to discuss it it's not trivial to turn an ip into more geo information

chronark avatar Oct 08 '24 16:10 chronark

@chronark I know it's not the same thing, but maybe we can use this: https://developers.cloudflare.com/ruleset-engine/rules-language/fields/standard-fields/#ipsrcsubdivision_2_iso_code ?

ogzhanolguncu avatar Oct 08 '24 19:10 ogzhanolguncu

@ogzhanolguncu do you mean showing the isocodes like GB-SWK?

maybe yeah, we still gotta get that information though. It's fairly easy to get on cloudflare, but as we're migrating our API over to aws, we need to figure out how to get that kind of data

chronark avatar Oct 08 '24 20:10 chronark

Okay, I see your point now. We can introduce 3rd party for the lookup, but that might be too costly on our end. We should definitely discuss 👍🏻

ogzhanolguncu avatar Oct 08 '24 20:10 ogzhanolguncu

/assign

mrjayantbokade avatar Oct 11 '24 04:10 mrjayantbokade