Intgrtns 559 whmcs ssl module v2.0.7 data truncation in csr generation approver email and host names empty incomplete - #8
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.
Tip: disable this comment in your organization's Code Review settings.
|
@claude review |
There was a problem hiding this comment.
Pull request overview
This PR updates SSL order creation in the Openprovider WHMCS SSL server module to build the order payload using a domain value sourced from service custom fields (instead of WHMCS’s $params['domain'], which can be empty for SSL products) and to handle wildcard domains when composing the approver email domain part.
Changes:
- Use
$params['customfields']['domain']to populatehost_name/host_namesand derive the email domain used forapprover_email. - Strip a leading
*.from the domain when generating the approver email’s domain part.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Fix SSL order creation to use the domain from custom fields instead of the empty WHMCS domain parameter. Also handles wildcard domains correctly when generating the approver email.