Enhance rune count in pkg/cmd/rollback.go and pkg/action/rollback.go

Signed-off-by: MrJack <36191829+biagiopietro@users.noreply.github.com>
MrJack 5 months ago
parent ff54bb328f
commit 53d3bedd26

@ -121,8 +121,8 @@ func (r *Rollback) prepareRollback(name string) (*release.Release, *release.Rele
return nil, nil, false, errInvalidRevision
}
if utf8.RuneCountInString(r.Description) > MaxDescriptionLength {
return nil, nil, false, fmt.Errorf("description must be %d characters or less, got %d", MaxDescriptionLength, utf8.RuneCountInString(r.Description))
if descLen := utf8.RuneCountInString(r.Description); descLen > MaxDescriptionLength {
return nil, nil, false, fmt.Errorf("description must be %d characters or less, got %d", MaxDescriptionLength, descLen)
}
currentReleasei, err := r.cfg.Releases.Last(name)

@ -68,8 +68,8 @@ func newRollbackCmd(cfg *action.Configuration, out io.Writer) *cobra.Command {
}
// Validate description length
if utf8.RuneCountInString(client.Description) > action.MaxDescriptionLength {
return fmt.Errorf("description must be %d characters or less, got %d", action.MaxDescriptionLength, utf8.RuneCountInString(client.Description))
if descLen := utf8.RuneCountInString(client.Description); descLen > action.MaxDescriptionLength {
return fmt.Errorf("description must be %d characters or less, got %d", action.MaxDescriptionLength, descLen)
}
dryRunStrategy, err := cmdGetDryRunFlagStrategy(cmd, false)

Loading…
Cancel
Save