Disallow non-file characters in pipeline names

[parkertimmins]

Pipeline names may currently contain several non-alpha characters, such as comma and asterisk. Pipelines with such names cannot be fetched by id from the GET _ingest/pipeline/ api as the comma is interpreted to mean multiple id queries, and the asterisk is treated as a star query. Thus these characters, along with other characters which are not allowed in index or alias names, should be disallowed from pipeline names.

closes https://github.com/elastic/elasticsearch/issues/104411

[elasticsearchmachine]

Pinging @elastic/es-data-management (Team:Data Management)

[elasticsearchmachine]

Hi @parkertimmins, I've created a changelog YAML for you.

[nielsbauman]

Should we consult the Backwards Compatability Committee (BCC) on this?

cc @dakrone

[dakrone]

This is definitely something we'd want to consider as breaking before merging (and should consult the BCC). Also (separately), I wouldn't consider this large enough to introduce a brand new exception, we could likely use IllegalArgumentException for this. We can use MetadataCreateIndexService.validateIndexOrAliasName for the validation and construction of the IllegalArgumentException (we do this in a few places in the code).

[parkertimmins]

I was thinking to discuss with the team whether or not this seemed breaking changy, but in that case I'll go ahead and open a BCC ticket.

[lukewhiting]

Closing this for now as it's been superseeded by https://github.com/elastic/elasticsearch/pull/114837