Skip to content

Exclude "/" from pipeline names too as it causes problems in URLs

Marek Szuba requested to merge experimental/no_slash_in_pipeline_name into version/2.4

Created by: muffato

Use case

Jorge found this one by accident. He had put a / in the pipeline_name option and then failed with this error Error: <stdin>: syntax error in line 119 near '/'

When doing my own tests, I realized that the URLs themselves would be confusing with a / in the database name, so better to remove them early on.


Just like whitespace, replace the / with an underscore

Possible Drawbacks



Have you added/modified unit tests to test the changes?

No, that function is not currently tested.

If so, do the tests pass/fail?

Have you run the entire test suite and no regression was detected?

Merge request reports