Fix SPARQL ASK queries to return W3C standard JSON format #42

Open

csajedi wants to merge 1 commits from csajedi/nextgraph-rs:fix/sparql-ask-json-format into master

csajedi commented 3 months ago

Description

This PR fixes ASK queries returning raw JavaScript booleans instead of the W3C SPARQL 1.1
standard JSON format.

Problem

ASK queries were returning true or false (raw booleans)
This violates the W3C SPARQL 1.1 specification
Made ASK queries incompatible with standards-compliant SPARQL clients

Solution

Use QueryResultsSerializer to format ASK results properly
Now returns {"head": {}, "boolean": true/false} as per W3C standard
Consistent with SELECT/CONSTRUCT query behavior

Testing

Built and tested locally with demonstration script
Verified both true and false cases return correct format
Confirmed SELECT queries continue to work as expected

References

W3C SPARQL 1.1 Query Results JSON Format: https://www.w3.org/TR/sparql11-results-json/

csajedi added 1 commit 3 months ago

8803c63946 fix: return W3C standard JSON format for SPARQL ASK queries

ASK queries were returning raw JavaScript booleans (true/false) instead of
the W3C SPARQL 1.1 standard JSON format {"head": {}, "boolean": true/false}.

This fix uses QueryResultsSerializer to properly format ASK query results,
ensuring compatibility with standards-compliant SPARQL applications and
consistency with SELECT/CONSTRUCT query behavior.

The issue was in ng-verifier/src/request_processor.rs where
AppResponseV0::True/False were returned instead of a properly
serialized QueryResult.

Fixes: Non-standard ASK query responses
Reference: https://www.w3.org/TR/sparql11-results-json/

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: NextGraph/nextgraph-rs#42