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

Expand on Neo4j and Query API #634

Open
wants to merge 29 commits into
base: console
Choose a base branch
from
Open

Conversation

fiquick
Copy link
Collaborator

@fiquick fiquick commented Mar 13, 2025

No description provided.

Copy link
Collaborator

@AlexicaWright AlexicaWright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some comments to straighten out.


Neo4j tools have traditionally connected over Bolt, a binary protocol that offers optimal performance and stability, that requires an open WebSocket connection to the Neo4j port.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

Is this true? I think neo4j has been around for quite some time and as mentioned above, it's built on bolt but it is not the same, so we should probably say that.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, I was curious about why the tooltip in the UI didn't mention Bolt - when a customer hovers over neo4j it says:

"Connect through the Neo4j protocol using a WebSocket connection to the Neo4j port. Use for best performance and stability."

Copy link
Collaborator

@AlexicaWright AlexicaWright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some more comments. Well done getting all this content together!

If you run into any issues with a particular request, you can https://support.neo4j.com/[raise a support ticket] and provide the `X-Request-Id`.

== Query API
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if the Query API belongs in the Aura API. We could maybe change the top level header to API and then have separate sections on Aura API and Query API.

To send Cypher queries to an Aura instance, make a POST request to the following endpoint:

https://<databaseID>.databases.neo4j.io/db/<databaseName>/query/v2
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be escaped, it shouldn't be clickable.

----

Note the last line above and inclusion of `:443` is not required as its the default.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then why is it included?

@recrwplay
Copy link
Contributor

This PR includes documentation updates
View the updated docs at https://neo4j-docs-aura-634.surge.sh

New pages:

Updated pages:

@recrwplay
Copy link
Contributor

This PR includes documentation updates
View the updated docs at https://neo4j-docs-aura-634.surge.sh

New pages:

Updated pages:

Copy link
Collaborator

@AlexicaWright AlexicaWright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some more comments.

* for Aura instances host is of the format `<databaseID>.databases.neo4j.io`

* the on-prem documentation refers to port `:7474` and this is true for a on-prem install but Aura, given its HTTPS, the <port> defaults as 443.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean with "on-prem"? This is not terminology we use externally.

In an enterprise environment with restrictive networking, connection errors are sometimes caused because non-standard ports used by the Neo4j protocol are blocked by firewalls.

== Query API for Aura
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is the spot for this either. We need to find a better place for it. Let's talk about it!

@neo-technology-commit-status-publisher
Copy link
Collaborator

This PR includes documentation updates
View the updated docs at https://neo4j-docs-aura-634.surge.sh

New pages:

Updated pages:

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.

4 participants