caddyserver / caddyserver/certmagic
DefaultCertificateSelector should log a debug message if no matching certificates are found.
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 5.6k
- Forks
- 354
- Avg merge
- 9d 23h
- Merged PRs (30d)
- 3
Description
What would you like to have changed?
DefaultCertificateSelector in handshake.go should log a debug message if no matching certificates are found.
There is currently just a comment for that case, reading:
// all matching certs are expired or incompatible, oh well
Why is this feature a useful, necessary, and/or important addition to this project?
It's
- Trivial to add.
- Having it would ease debugging - I spent two hours wondering why things weren't working because I didn't bother adding SANs to my test certificates, and a debug message would have helped me find the problem sooner.
What alternatives are there, or what are you doing in the meantime to work around the lack of this feature?
I don't think there's any meaningful alternatives or workarounds.
Please link to any relevant issues, pull requests, or other discussions.
I have documented my mistakes so that others needn't repeat them. https://github.com/caddyserver/caddy/issues/5450
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in handshake.go at DefaultCertificateSelector, especially the branch associated with the comment that all matching certificates are expired or incompatible. Add the requested debug logging there and verify that the message is emitted when no matching certificates are found.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- security
- Issue type
- Feature
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100