Should honor timezone information on requests and events

[roccoblues]

During a migration I had the (unfortunate) situation that some logjam agents where sending requests and events in UTC time and some in CET. (See also: https://github.com/skaes/logjam_agent/issues/32).

It looks like the data is stored with the timezone information in the logjam database but the view doesn't handle it correctly.

Ideally logjam would adjust all times to the timezone configured for the logjam app (Rails.application.config.time_zone) and also adjust the graphs to it.

[skaes]

Actually it is the logjam-importer which needs to be changed. It would need to convert incoming timestamps in all sorts of places to the desired timezone.

BTW: calling Rails.application.config.time_zone gives "UTC", even though running date on the console returns "Mon Oct 30 11:44:29 CET 2017"

[roccoblues]

That's because Rails.application.config.time_zone defaults to UTC.

What I mean is that you should store all data in UTC (or with timezone information) in the database an then convert it for the view layer to Rails.application.config.time_zone. That way the user could choose the correct display timezone for his location.

But I guess that will be hard to do because you store daily databases.

[roccoblues]

I know it's complicated and (at least for me) currently not required but as soon as you have servers with different timezone settings logjam will get confused. So don't let XING setup anything in the US. :-)