Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AKHQ must handle datetime with time zone consistently everywhere #1400

Closed
wnleao opened this issue Mar 9, 2023 · 0 comments
Closed

AKHQ must handle datetime with time zone consistently everywhere #1400

wnleao opened this issue Mar 9, 2023 · 0 comments

Comments

@wnleao
Copy link

wnleao commented Mar 9, 2023

The way AKHQ handles datetime with time zones must be coherent by using the same time zone every place it handles datetime on the UI (e.g. when presenting dates and when handling timestamp selection).

We suggest that AKHQ should use UTC by default and in a future work the user could define a different time zone in the settings.

Topic filter timestamp

AKHQ should also be explicit about which timezone is being used to represent the date when selecting the timestamp. For instance, on the image below, the Date columns show the date using Zulu, but the javascript component is using the browser’s timezone to convert the selected timestamp and filter the data.

Screenshot 2023-03-09 at 14 50 40

Consumer group update offsets

When updating offsets (see image below), it should be explicit to the user which time zone is going to be used (UTC by default).

Screenshot 2023-03-09 at 14 45 23

When one selects a date and time, AKHQ is using in the request the same provided timestamp as UTC (see image below), which is the expected behavior.

Screenshot 2023-03-09 at 14 46 02

We suggest that the Timestamp label should be “Timestamp (UTC)” or the field value should append the suffix like “08-03-2023 10:08 Z” or “08-03-2023 10:08 +00:00”.

Produce to topic

When producing a message to a topic, currently, the provided timestamp is converted to UTC without an explicit representation on the UI. I'm working with CET here (1 hour ahead of UTC).

Screenshot 2023-03-09 at 14 46 54

One can confirm that the request payload is using the UTC on the image below.

Screenshot 2023-03-09 at 14 49 07

We suggest that the timestamp should be handled as UTC and:

  • the timestamp label should be “Timestamp (UTC)”; or
  • the field value should append the suffix like “08-03-2023 10:08 Z” or “08-03-2023 10:08 +00:00”.

Let us (@flysen and I) know if you think that this is something worth pursuing and we would be more than glad to help.

tchiotludo pushed a commit that referenced this issue Apr 4, 2023
close #1400

Co-authored-by: Fredrik Lysén <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant