per-johnsson-aurora opened a new issue, #1267: URL: https://github.com/apache/directory-scimple/issues/1267
A SCIM resource is created at a distinct point of time but its representation `org.apache.directory.scim.spec.schema.Meta#created`) is represented as a `LocalDateTime`, which do not itself correspond to specific point in time. Same goes for `org.apache.directory.scim.spec.schema.Meta#lastModified`. Although this format seems valid according to RFC 7643 (timezone part is optional), this is a likely cause of errors due to `LocalDateTime` ambiguity and there is also a risk that a client uses `org.apache.directory.scim.spec.resources.ScimUser#timezone` for disambiguation which would not work very well (timezone might be null/timezone might change etc). To me, these fields would be much better represented as `java.time.Instant`s (formatted with `DateTimeFormatter.ISO_INSTANT`). If the `LocalDateTime`s are expected to contain an implicit UTC timezone then it should be documented. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
