Skip to content

Add support for SiteEncrypt.Phoenix.Endpoint detection#339

Merged
zachdaniel merged 2 commits intoash-project:mainfrom
Hermanverschooten:support-siteencrypt-phoenix-endpoint
Oct 14, 2025
Merged

Add support for SiteEncrypt.Phoenix.Endpoint detection#339
zachdaniel merged 2 commits intoash-project:mainfrom
Hermanverschooten:support-siteencrypt-phoenix-endpoint

Conversation

@Hermanverschooten
Copy link
Copy Markdown
Contributor

Enhanced endpoint detection functions to support SiteEncrypt.Phoenix.Endpoint in addition to Phoenix.Endpoint. This allows Igniter to work with projects that use SiteEncrypt for automatic SSL certificate management.

Changes:

  • Updated select_endpoint/3 to detect both Phoenix.Endpoint and SiteEncrypt.Phoenix.Endpoint

  • Updated endpoints_for_router/2 to detect both endpoint types

  • I accept the AI Policy.

  • Bug fixes include regression tests

  • Chores

  • Documentation changes

  • Features include unit tests

  • Refactoring

  • Update dependencies

Enhanced endpoint detection functions to support SiteEncrypt.Phoenix.Endpoint
in addition to Phoenix.Endpoint. This allows Igniter to work with projects
that use SiteEncrypt for automatic SSL certificate management.

Changes:
- Updated select_endpoint/3 to detect both Phoenix.Endpoint and SiteEncrypt.Phoenix.Endpoint
- Updated endpoints_for_router/2 to detect both endpoint types
- Added comprehensive tests for SiteEncrypt.Phoenix.Endpoint support
Comment thread lib/igniter/libs/phoenix.ex Outdated
As suggested in PR review, refactored the endpoint detection to use a
module attribute @endpoint_modules and Enum.find_value/2 to check for
matching endpoint types. This makes it easier to add more endpoint
types in the future and reduces code duplication.
@zachdaniel zachdaniel merged commit 6153ea0 into ash-project:main Oct 14, 2025
@zachdaniel
Copy link
Copy Markdown
Contributor

🚀 Thank you for your contribution! 🚀

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