Skip to content

Experimental HTTP support via transactional Cypher endpoint #326

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 5 commits into from
Feb 20, 2018

Conversation

lutovich
Copy link
Contributor

@lutovich lutovich commented Feb 1, 2018

PR adds a driver implementation that use REST API. It now only supports Session#run(), not explicit transactions or routing. Driver will only expose native numbers because it receives JSON which can't represent large integers. Behaviour is similar to Bolt driver configured with {disableLosslessIntegers: true}.

This is an experimental feature and should under no circumstances be used in production environment!

Based on #323

Commit adds a driver implementation that use REST API. It now only
supports `Session#run()`, not explicit transactions or routing. Driver
will only expose native numbers because it receives JSON which can't
represent large integers. Behaviour is similar to Bolt driver
configured with `{disableLosslessIntegers: true}`.

This is an experimental feature and should under no circumstances
be used in production environment!
@lutovich lutovich force-pushed the 1.6-http-experimental branch from aa93910 to 1015759 Compare February 12, 2018 16:30
Driver created with either HTTP or HTTPS url needs to execute REST
requests. This commit makes it use new standard Fetch API instead of
3rd party XHR library, which is based on `XMLHttpRequest`.
For driver created with either `http` or `https` url.
@lutovich lutovich force-pushed the 1.6-http-experimental branch from 628df93 to 293c23a Compare February 19, 2018 15:15
@lutovich lutovich merged commit 8fb84d2 into neo4j:1.6 Feb 20, 2018
@lutovich lutovich deleted the 1.6-http-experimental branch February 20, 2018 16:22
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.

1 participant