Do Asset ID's *need* to be sanatized?

[akinnane]

This function rejects asset IDs that conform to our own resource naming conventions. Is there a good reason why this check exists, and why I shouldn't disable this code for our assets?

threagile/pkg/model/parse.go

Lines 755 to 761 in 49f47fb

	func checkIdSyntax(id string) error {
	validIdSyntax := regexp.MustCompile(`^[a-zA-Z0-9\-]+$`)
	if !validIdSyntax.MatchString(id) {
	return fmt.Errorf("invalid id syntax used (only letters, numbers, and hyphen allowed): %v", id)
	}
	return nil
	}

[ezavgorodniy]

I don't know the original idea @cschneider4711 had when this code was added but I'd say "id is id" and currently tool is providing a way to add non sanitized and more human friendly into name and description, while id is used in many places for references purpose.