Skip to content

Raise ConfigurationError on direct driver with routing context - #1178

Merged
robsdedude merged 2 commits into
neo4j:6.xfrom
robsdedude:raise-on-direct-driver-with-routing-ctx
May 16, 2025
Merged

Raise ConfigurationError on direct driver with routing context#1178
robsdedude merged 2 commits into
neo4j:6.xfrom
robsdedude:raise-on-direct-driver-with-routing-ctx

Conversation

@robsdedude

Copy link
Copy Markdown
Member

Raise ConfigurationError instead of ignoring the routing context (URI query parameters) when creating a direct driver ("bolt[+s[sc]]://" scheme). A deprecation warning was already being emitted in such case.

@robsdedude
robsdedude force-pushed the raise-on-direct-driver-with-routing-ctx branch from 2a53e2e to 6717f9a Compare May 14, 2025 15:16

@AndyHeap-NeoTech AndyHeap-NeoTech left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@robsdedude
robsdedude enabled auto-merge (squash) May 16, 2025 17:03
@robsdedude
robsdedude merged commit 66ff306 into neo4j:6.x May 16, 2025
@robsdedude
robsdedude deleted the raise-on-direct-driver-with-routing-ctx branch May 16, 2025 18:00
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