feature: timezone support for graylog

[crcro]

DO NOT DELETE THIS TEXT

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

• Have you followed our code guidelines?

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926

feature request from community site

before:

after:

[murrant]

Not sure I like the way this is fixed. I think they should just be displayed in the timezone of the browser accessing them. Which means the translation should probably be done in javascript with moment.js

[murrant]

http://www.jquery-bootgrid.com/documentation#converters

[crcro]

but in that case won't it be forced to user? maybe there are some that still would like to get the exact data as graylog stores them

[murrant]

If you don't want to force it on the user, you should add a timezone setting to the user settings. With browser timezone as the default.

That is a much larger scope. Up to you.

[crcro]

@murrant: so I made it like:

if no timezone is set in config, print default user timezone using moment.js.
if timezone is set then display using php date functions.

is this better?

[laf]

Any reason why you've removed my name from the copyright?

[crcro]

By mistake, copy/paste the entire copyright section, restored it. Sorry

[murrant]

Yeah, seems a like a good compromise for now. Thanks.

[laf]

Thanks for this. One little change from me.

I've also posted in the original issue for testers. If we don't hear from someone in a few days then I'm happy to merge in.

[laf]

Really petty but this should still be LibreNMS

[scrutinizer-notifier]

The inspection completed: No new issues

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed.