Skip to content

Fix/bunny zone record ids - #9

Open
cseufert wants to merge 3 commits into
getnamingo:mainfrom
moddengine:fix/bunny-zone-record-ids
Open

Fix/bunny zone record ids#9
cseufert wants to merge 3 commits into
getnamingo:mainfrom
moddengine:fix/bunny-zone-record-ids

Conversation

@cseufert

Copy link
Copy Markdown

Sorry this PR is a little messy as it updates dependencies as well as adding some features.

  • Bunny DNS Upgrades: Enhanced Bunny provider to return record IDs, improve zone ID caching, and support targeting records directly by ID.
  • Zone ID Tracking: Updated createDomain in Service.php to store and sync zoneId across MySQL, PostgreSQL, and SQLite.
  • Record Key Fixes: Standardized database queries in Service.php and Hetzner.php to target internal ids for database rows while using recordId for provider API calls.
  • Extended Record Metadata: Added explicit support for SRV and advanced DNS parameters (weight, port, flags, tag) during record modifications.
  • Dependencies & Tooling: Bumped guzzlehttp/guzzle to ^7.15.2, added .gitignore rules for /vendor/, and added a PHP 8.3 shell.nix environment.

AI was used in developing these changes.

@cseufert

Copy link
Copy Markdown
Author

@getpinga Is it possible to get this merged, I have some PR's I want to open for whmcs-dns, but it would be better if those patches were against the official plexdns pkg and not my fork.

@getpinga

Copy link
Copy Markdown
Contributor

Thank you, will be merged today or tomorrow. Sorry for delay.

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