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

Turtle files should not be parsed and re-serialized on GET #60

Open
timbl opened this issue Dec 14, 2015 · 5 comments
Open

Turtle files should not be parsed and re-serialized on GET #60

timbl opened this issue Dec 14, 2015 · 5 comments

Comments

@timbl
Copy link
Member

timbl commented Dec 14, 2015

Turtle files should not be parsed and re-serialized (unless you have to convert format) on GET because:

  • It takes time to do,
  • people also may want to save a file including comments
  • people may want to save a file and note its hash or sign it.

Example: This file uses a namepsace for foaf: in the original:

$ curl https://timbl.databox.me/solid/issues/state.ttl
@Prefix rdf: http://www.w3.org/1999/02/22-rdf-syntax-ns# .

<>
http://xmlns.com/foaf/0.1/principalSubject <#this> .

$

@csarven
Copy link
Member

csarven commented Dec 14, 2015

I mentioned this in gitter/gold

When I order the workspaces in my preferences file, and PUT it back, and then GET, the prefs again, it appears to be alphabetically reordered. Tested on databox.me.

@deiu confirmed that it will be changed.

@deiu
Copy link
Contributor

deiu commented Dec 14, 2015

@csarven that doesn't really matter. Turtle does not guarantee you that triples will be ordered.

@csarven
Copy link
Member

csarven commented Dec 14, 2015

@deiu I'm strictly talking about the file level. While the order is not guaranteed for RDF formats, is there a UC why gold needs to take extra step to re-serialize instead of just serving the file?

@deiu
Copy link
Contributor

deiu commented Dec 14, 2015

No, it's just the way it was implemented a while ago. It will not sniff in the future versions.

@akuckartz
Copy link

But clients can not trust that the serialisation does not change as long as this behavior is not specified...

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

4 participants