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

[BUG] XDM xs:dateTimeStamp error on eXist-db 6.x.x #21

Closed
adamretter opened this issue Aug 25, 2023 · 0 comments · Fixed by #22
Closed

[BUG] XDM xs:dateTimeStamp error on eXist-db 6.x.x #21

adamretter opened this issue Aug 25, 2023 · 0 comments · Fixed by #22
Labels
bug Something isn't working

Comments

@adamretter
Copy link
Member

adamretter commented Aug 25, 2023

When trying to use the current version on eXist-db 6.x.x the following error is reported:

HTTP ERROR 500 jakarta.servlet.ServletException: jakarta.servlet.ServletException: An error occurred while processing request to /exist/apps/myapp/index.html: err:XPTY0004 xs:dateTimeStamp(2023-08-25T14:22:00.502-05:00) is not a sub-type of xs:string [at line 114, column 49, source: /db/apps/existdb-saml/content/exsaml.xqm] In function: exsaml:store-authnreqid(xs:string, xs:string) [114:19:/db/apps/existdb-saml/content/exsaml.xqm] exsaml:build-saml-authnreq() [92:17:/db/apps/existdb-saml/content/exsaml.xqm] exsaml:build-authnreq-redir-url(xs:string) [53:29:/db/apps/existdb-saml/content/exsaml.xqm]

This occurs because fn:dateTime has been updated to be in compliance with the newer XQuery 3.1 definition, which causes it to return as xs:dateTimeStamp, this cannot be simply downcast to an xs:string which is what the function exsaml:store-authnreqid expects.
Instead to fix this (whilst maintaining backwards compatibility) the value should be passed around as an xs:dateTime.

@adamretter adamretter added the bug Something isn't working label Aug 25, 2023
@adamretter adamretter changed the title [BUG] Current version won't run on eXist-db 6.x.x [BUG] XDM xs:dateTimeStamp error on eXist-db 6.x.x Aug 25, 2023
@dizzzz dizzzz closed this as completed in #22 Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant