Skip to content

2025.x changes #330

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

Merged
merged 2 commits into from
Aug 12, 2025
Merged

2025.x changes #330

merged 2 commits into from
Aug 12, 2025

Conversation

vga91
Copy link
Collaborator

@vga91 vga91 commented Jul 29, 2025

Changes list

  • In RDFEndpoint.java changed @Path("/{dbname}/describe/{nodeidentifier}") to @Path("/{dbname}/describe")
    It's not possible anymore to use the endpoint as before, because it throws HTTP ERROR 400 Ambiguous URI path separator.
    This is due to neo4j 5.x uses jetty-server 10.x, as we can se here, while neo4j 2025.x uses jetty-server (see here).
    The jetty 12.x doesn't permit by default an inner endoded URL, e.g./rdf/<dbName>/describe/http%3A%2F%2Fneo4j.org%2Find%23neo4j355, like explained here for example.
    So, even if is a breaking change, that we need to change nodeidentifier from @PathParam to @QueryParam and calling it in this way /rdf/<dbName>/describe?nodeidentifier=<URI>.
    Changed the docs about it where needed.

  • Changed RDFEndpointTest

    • Changed neo4j/describe/{nodeIdentifier} to neo4j/describe?nodeIdentifier=
  • Changed RDFProcedureTest

    • The test multivalMultitypeSamePartialTx was flaky since it took the 1st node "Resource" that sometimes was not the correct one to be tested
  • Changed pom.xml

    • Upgraded to java 21, used by neo4j 2025.x

    • Changed antora.yml

  • Various docs changes

    • Changed call dbms.procedures() to show procedures
    • Added appendix with the 2025.x changes

Tests executed

Tested with neo4j 2025.06 enterprise and community.
Tested with and without apoc jars.

:POST /rdf/neo4j2/cypher
{ "cypher" : "MATCH path = (n:Order { orderID : '10785'})-[:ORDERS]->()-[:PART_OF]->(:Category { categoryName : 'Beverages'}) RETURN path " , "format": "RDF/XML" , "mappedElemsOnly" : true }
:POST http://localhost:7476/rdf/neo4j2/cypher
{ "cypher" : "MATCH path = (n:Order { orderID : '10785'})-[:ORDERS]->()-[:PART_OF]->(:Category { categoryName : 'Beverages'}) RETURN path " , "format": "RDF/XML" , "mappedElemsOnly" : true }
:GET /rdf/ping
:GET http://localhost:7476/rdf/neo4j/describe?nodeIdentifier=http%3A%2F%2Fneo4j.com%2Fmovies%2FKeanu

@vga91 vga91 requested a review from jexp July 29, 2025 09:54
@jbarrasa jbarrasa merged commit 683c8c9 into 2025.06 Aug 12, 2025
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

Successfully merging this pull request may close these issues.

2 participants