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

Expose ParserOption on more places #2529

Open
ajuvercr opened this issue Oct 15, 2024 · 1 comment · May be fixed by #2530
Open

Expose ParserOption on more places #2529

ajuvercr opened this issue Oct 15, 2024 · 1 comment · May be fixed by #2530
Labels
bug Something isn't working

Comments

@ajuvercr
Copy link

Search terms you've used

  • ParserOption
  • getSolidDatasetWithAcl

Bug description

It is currently not possible to add parser options when getting a solid dataset with acl.
This is problematic when the resource is not text/turtle but something like application/trig.

Expected result

getSolidDatasetWithAcl returns the expected dataset with acl options without crashing.

Actual result

getSolidDatasetWithAcl crashes with a parse error.

Additional information

Note that I don't expect the solid-client to handle graphs correctly, as I understand it no graphs are introduced anywhere.
My use case is to use this function to get and update the acl files for a trig resource.

@ajuvercr ajuvercr added the bug Something isn't working label Oct 15, 2024
@ajuvercr ajuvercr linked a pull request Oct 15, 2024 that will close this issue
1 task
@NSeydoux
Copy link
Contributor

Hello @ajuvercr , thanks for opening (and proposing a fix) for this! I'll review the PR right away, sorry for the delay.

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.

2 participants