Skip to content

Add FQDN to API URL and update version to 2.3.2#119

Merged
cayossarian merged 2 commits intomainfrom
2_3_2
Mar 19, 2026
Merged

Add FQDN to API URL and update version to 2.3.2#119
cayossarian merged 2 commits intomainfrom
2_3_2

Conversation

@cayossarian
Copy link
Member

No description provided.

@cayossarian cayossarian requested a review from Copilot March 19, 2026 02:23
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds client helpers for managing a SPAN Panel’s registered TLS FQDN (for certificate SAN inclusion) and bumps the project version metadata.

Changes:

  • Add v2 DNS FQDN management helpers: register_fqdn(), get_fqdn(), delete_fqdn() targeting /api/v2/dns/fqdn
  • Export the new helpers from the package top-level API
  • Bump pyproject.toml version to 2.3.2 and add a changelog entry describing the feature

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
src/span_panel_api/auth.py Adds async HTTP helpers for registering/retrieving/deleting the panel TLS FQDN.
src/span_panel_api/__init__.py Exposes the new FQDN helpers via top-level imports and __all__.
pyproject.toml Updates distribution version to 2.3.2.
CHANGELOG.md Documents the new FQDN endpoints (currently under a 2.4.0 heading).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Align __version__ and CHANGELOG heading with pyproject.toml 2.3.2.
Add unit tests for register_fqdn, get_fqdn, delete_fqdn covering
success codes, auth errors, API errors, connection errors, and
timeouts. Add export test for the new FQDN symbols.
@cayossarian cayossarian merged commit 903ab60 into main Mar 19, 2026
7 checks passed
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