Skip to content

Validation of LD-files #882

Closed
Closed

Description

@megoth

When posting triples as TTL, RDF+XML, JSON-LD, etc, I think we should do validation on the server. I don't think this should be to hard to implement, but the devils probably in the detail, so want to have a good discussion on this.

Also, I'm uncertain if this should be part of 5.0.0. or v.next. Any thoughts?

Activity

  1. kjetilk commented on Oct 23, 2018

    @kjetilk
    Member

    I think it is clearly a part of the v.next architecture, but it seems to touch all the handlers in the current architecture.

  2. akuckartz commented on Oct 23, 2018

    @akuckartz

    What happens when there is no validation?

  3. kjetilk commented on Oct 23, 2018

    @kjetilk
    Member

    The server would respond with a 400. It is no replacement for client-side validation, but it is a last stand against breaking stuff.

  4. megoth commented on Oct 24, 2018

    @megoth
    ContributorAuthor

    I also think this would limit some of the issues that people have experienced (solid/solid#165, solid/solid#206, #826)

  5. added this to the 5.0.0 milestone on Nov 6, 2018
  6. self-assigned this
    on Nov 9, 2018
  7. megoth commented on Nov 14, 2018

    @megoth
    ContributorAuthor

    I'm discontinuing my work on this until we have a broader consensus on validation in general (currently discussed in Validation and filter feature and Discourse).

    I've pushed my work on the branch feature/simple-validation, in case it's interesting to continue at some point.

  8. megoth commented on Nov 14, 2018

    @megoth
    ContributorAuthor

    After a discussion with @timbl, @kjetilk, and @rubensworks on this earlier today it seems that validation (i.e. to make sure that the Turtle syntax is valid) on the ACL-files might be a good compromise. If noone objects to this, I'll continue working on that tomorrow (it's should be almost done in that regard).

  9. megoth commented on Feb 4, 2019

    @megoth
    ContributorAuthor

    This issue has been fixed, we implemented syntax validation for ACL files in #937

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions