Repository navigation
Replies: 1 comment
|
I believe this has been resolved in 26.2.3 with 08704df. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
GET /admin/realms/{realm}/eventsandGET /admin/realms/{realm}/admin-eventsallow events to be filtered by a date using query parameters. Currently, those query strings expect provided dates to be provided in theyyyy-MM-ddformat. However, this format does not allow precise time filtering, and may also cause confusion due to lack of timezone information. Furthermore, the expected format is not listed in the documentation at any point and can only be found through an error returned by the events endpoint or by reading source code.Ideally, it would be great to use a more precise format for date filtering. A unix timestamp would be the format to make sense, as that is the format used for the time field in the event response (and elsewhere within keycloak). Of course, a fallback to
yyyy-MM-ddshould be kept for backwards compatibility.I'm not incredibly familiar with java, but I would be interested in contributing this change if it's something that would be seen a beneficial. If a change in the date format isn't wanted, I would be happy to at least send a PR to update the documentation with the expected date format.
In addition to changes on the API side, changes would be required both in the API and in the admin UI. A beta component is available in the PatternFly react-core components for a combined date and time picker, though it may also be acceptable stick with the plain date picker within the UI, and simply convert to unix timestamp before sending the request to prevent any issues with possible timezone differences.
All reactions