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

Add Bolt API #832

Merged
merged 8 commits into from
Mar 11, 2025
Merged

Add Bolt API #832

merged 8 commits into from
Mar 11, 2025

Conversation

injectives
Copy link
Contributor

No description provided.

@injectives injectives marked this pull request as draft January 24, 2025 14:41
@michael-simons
Copy link
Collaborator

Net negative LoC => LGTM ;)

@michael-simons
Copy link
Collaborator

I went through all of this, rebased on main, added the tests back in that have been just commented out, fixed the bundle builds and I'd say this is good to go when we have the bits on central.

@michael-simons
Copy link
Collaborator

@injectives please have a last look, release the module and than we can merge.

This is now in an acceptable state for me. I isolated all the bolt stuff proper, and decouple the value package correctly

@michael-simons michael-simons marked this pull request as ready for review March 4, 2025 18:52
@michael-simons michael-simons force-pushed the bolt-api branch 2 times, most recently from d2fd0d6 to 6a824a8 Compare March 7, 2025 11:42
injectives and others added 6 commits March 10, 2025 15:07
Use Bolt API AuthToken

Move to an updated BoltConnectionProvider interface

Add SecurityPlan support

Add network timeout support

Handle error in setting network timeout

Restore BoltAgentUtilTests

Restore ProductVersionTests

Update RecordImpl visibility

Revert pom.xml update

Update PackageStructureTests

Update to latest driver, fix bundle builds.

fix log.

repackage bolt adapters

add tests back in

f

f
@michael-simons michael-simons merged commit 0306bf1 into neo4j:main Mar 11, 2025
5 checks passed
@injectives injectives deleted the bolt-api branch March 11, 2025 12:19
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