Skip to content

Add ReadOnlyIntent connection option to Lite#515

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/readonly-intent
Mar 11, 2026
Merged

Add ReadOnlyIntent connection option to Lite#515
erikdarlingdata merged 1 commit intodevfrom
feature/readonly-intent

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • Adds ReadOnlyIntent bool property to Lite's ServerConnection model (persisted to servers.json, defaults to false)
  • Sets ApplicationIntent=ReadOnly on the connection string when enabled
  • Adds checkbox in Add/Edit Server dialog: "Read-only intent (for AG listeners and readable replicas)"
  • Test Connection button also respects the setting

Why

Required for users connecting Lite via:

  • AG listeners — routes to readable secondary
  • Azure SQL failover group endpoints — same listener-based routing
  • Business Critical / Managed Instance built-in read replicas (only reachable via intent flag)

Dashboard excluded since it requires a writable PerformanceMonitor database.

Test plan

  • Add a server without the checkbox — verify connection string has no ApplicationIntent change
  • Add a server with the checkbox — verify ApplicationIntent=ReadOnly in connection string
  • Edit an existing server, toggle the checkbox, save — verify persistence
  • Test Connection works with the checkbox enabled against a readable replica

🤖 Generated with Claude Code

Adds a checkbox in the Add/Edit Server dialog to set ApplicationIntent=ReadOnly
on the connection string. Required for connecting via AG listeners, Azure SQL
failover groups, and Business Critical / Managed Instance built-in read replicas.

Dashboard excluded — it requires a writable PerformanceMonitor database.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 757b264 into dev Mar 11, 2026
3 checks passed
@erikdarlingdata erikdarlingdata deleted the feature/readonly-intent branch April 9, 2026 00:33
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