Add TODO for domain label values

This commit is contained in:
James Munnelly 2018-04-04 11:30:15 +01:00
parent 01efbca114
commit b0e65f84c7

View File

@ -17,6 +17,10 @@ import (
func podLabels(crt *v1alpha1.Certificate, domain string) map[string]string {
return map[string]string{
certNameLabelKey: crt.Name,
// TODO: we need to support domains longer than 63 characters
// this value should probably be hashed, and then the full plain text
// value stored as an annotation to make it easier for users to read
// see #425 for details: https://github.com/jetstack/cert-manager/issues/425
domainLabelKey: domain,
}
}