docs: add certforge-issuer to external issuers listUpdate issuers.md#2118
docs: add certforge-issuer to external issuers listUpdate issuers.md#2118CertForge100 wants to merge 2 commits into
Conversation
Propose changes Signed-off-by: CertForge100 <admin@certforge.xyz>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Deploy Preview for cert-manager ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Adds certforge-issuer to the cert-manager external issuers documentation list, including the required reference links for docs, CA/provider site, and release page.
Changes:
- Added a new 🥈 tier table entry for
certforge-issuer. - Added reference link definitions for
config:certforge-issuer,ca:certforge-issuer, andrelease:certforge-issuer.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Thank you for submitting this. A few items need addressing before this can move forward.
CI failure
The spelling check fails — certforge needs to be added to .spelling. (See #2130 for a recent example where solidserver was added.)
Formatting
The PR removes blank lines between the reference link sections (between [config:...] and [//]: # (CA docs), and between [ca:...] and [//]: # (Release pages)). These section separators should be preserved — compare with the existing file structure.
PR title
The title appears to have a concatenation error: ...external issuers listUpdate issuers.md — "listUpdate" should be two separate words or just "list".
Project maturity
The certforge-issuer repository was created on 20 May 2026 (less than a month ago) with 0 stars, 0 forks, 15 commits, and no open-source licence. The first release (v0.1.0) was cut on 22 May and this PR was opened on 29 May — the same day as the v0.1.1 release.
While the 🥈 tier criteria only require "a release in the last 12 months" (which is technically met), the project has no licence file, which means the code is legally all-rights-reserved. That is a concern for listing it as a cert-manager integration — users cannot legally use or modify the code without an explicit licence grant.
Non-functional product links
The PR adds a [ca:certforge-issuer] link pointing to certgovernance.app. I checked the linked product infrastructure:
- Docs (
docs.certforge.xyz) — returns HTTP 403 Forbidden - Signup (
app.certforge.xyz/signup) — connection refused (nothing listening) - Self-hosted repo (
github.com/certforge/self-hosted, linked from the landing page) — returns 404, repository does not exist
The issuer is a proxy to this SaaS product, which does not appear to be operational. Listing it on the cert-manager website would direct users to a product they cannot actually use.
Could you:
- Add an open-source licence to the certforge-issuer repository
- Ensure the product links are functional before requesting a listing
- Fix the CI failure (
.spelling) - Restore the blank line separators between reference link sections
- Fix the PR title
|
Hey. Nothing to do with your PR, but I've noticed some dead links on https://certgovernance.app/:
Update: links have been updated, they all work now. |
Adds certforge-issuer to the list of known external cert-manager issuers.
What is certforge-issuer?
certforge-issuer bridges cert-manager to CertForge's policy engine. It adds governance controls cert-manager doesn't provide natively:
Existing cert-manager
Certificateresources require no changes — certforge-issuer is added as theissuerRefand governance is in place immediately.Links
Tier: 🥈 — released within 12 months, open source