Add --base-url flag for connector testability

[robert-chiniquy]

Summary

Adds the --base-url CLI flag to enable overriding the default API endpoint for testing purposes.

Changes

• Added BaseURLField to configuration
• Updated client/connector to accept and use the base URL override
• When --base-url is provided, it takes precedence over the default API URL

Files Modified

cmd/baton-cloudflare-zero-trust/config.go,cmd/baton-cloudflare-zero-trust/main.go,pkg/connector/connector.go

Testing

The connector can now be tested against mock servers:

baton-cloudflare-zero-trust --base-url http://localhost:8080 [other-flags]

Related

Part of the Connector Testability initiative.

Summary by CodeRabbit

• New Features
  • Added support for configuring a custom API endpoint base URL, enabling connection to alternate API environments.

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)

• cmd/baton-cloudflare-zero-trust/config.go is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

Walkthrough

The New function in the connector package now accepts an additional baseURL parameter and conditionally passes it to the underlying Cloudflare client initialization through variadic options when a non-empty baseURL is provided.

Changes

Cohort / File(s)	Summary
Connector Configuration pkg/connector/connector.go	Added baseURL parameter to the New function signature. Function now builds a local options slice and conditionally appends cloudflare.BaseURL(baseURL) before passing options to both API key and API token initialization paths.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A baseURL hops through the gate,
Our connector, now more ornate,
Options flow where once were few,
Flexible paths, both old and new! 🌿

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding a --base-url flag to improve connector testability.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch rch/testability/fix-base-url

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[russellhaering]

LGTM - base-url flag implementation looks correct. Cleanest approach of the batch -- leverages cloudflare-go SDK's native BaseURL option, correctly applied to both API key and API token auth paths.