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

sparqlEscapeDate omits timezone offset part #40

Open
MikiDi opened this issue May 17, 2022 · 0 comments
Open

sparqlEscapeDate omits timezone offset part #40

MikiDi opened this issue May 17, 2022 · 0 comments

Comments

@MikiDi
Copy link
Contributor

MikiDi commented May 17, 2022

Omitting the timezone offset can be interpreted here as "effectively changing the timestamps' value", since timestamps with no timezone offset specified are assumed to be local .
for ref http://books.xmlschemata.org/relaxng/ch19-77041.html

Similar mistake was made and corrected here. Note though that in the referenced implementation it is about client-side serialization of a date, while here it's about server side-serialization. Given the shortcomings of native JS Date localization options and the fact that servers are mostly configured with an UTC timezone, the approach used is in the transform-helpers, probably wouldn't suffice here.

@MikiDi MikiDi changed the title sparqlEscapeDate omits timezone part sparqlEscapeDate omits timezone offset part Jun 30, 2022
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