Add config to chained validator

[Richard87]

No description provided.

[satr]

Optional suggestion - to make it easier to read

Suggested change

	for index, validator := range v.validators {
	principal, err = validator.ValidateToken(ctx, token)
	if err == nil {
	return principal, nil
	} else if index == len(v.validators)-1 {
	return nil, err
	}
	}
	if principal, err = validator.ValidateToken(ctx, token); err == nil {
	return principal, nil
	} else if index == len(v.validators)-1 {
	return nil, err
	}

or

Suggested change

	for index, validator := range v.validators {
	principal, err = validator.ValidateToken(ctx, token)
	if err == nil {
	return principal, nil
	} else if index == len(v.validators)-1 {
	return nil, err
	}
	}
	principal, err = validator.ValidateToken(ctx, token)
	if err == nil {
	return principal, nil
	}
	if index == len(v.validators)-1 {
	return nil, err
	}

or

Suggested change

	for index, validator := range v.validators {
	principal, err = validator.ValidateToken(ctx, token)
	if err == nil {
	return principal, nil
	} else if index == len(v.validators)-1 {
	return nil, err
	}
	}
	var principal TokenPrincipal
	var err error
	for _, validator := range v.validators {
	if principal, err = validator.ValidateToken(ctx, token); err == nil {
	return principal, nil
	}
	}
	if err != nil {
	return nil, err
	}

[Richard87]

fixed with:

func (v *ChainedValidator) ValidateToken(ctx context.Context, token string) (TokenPrincipal, error) {
	var errs []error

	for _, validator := range v.validators {
		principal, err := validator.ValidateToken(ctx, token)
		if principal != nil {
			return principal, nil
		}
		errs = append(errs, err)
	}

	return nil, fmt.Errorf("%w: %v", errNoIssuersFound, errors.Join(errs...))
}

[satr]

No need for names in returned values - method returns them explicitly, like return principal, nil or return nil, err

[Richard87]

agreed, returning a list of errors if not found :)