search

microsoft / ga4gh-tes

C# implementation of the GA4GH TES API; provides distributed batch task execution on Microsoft Azure
MIT License
32 stars 26 forks source link

Empty/whitespace shall have same effect as null in Tes.Input.URL / Content #742

Closed MattMcL4475 closed 1 month ago

MattMcL4475 commented 1 month ago

Describe the bug If a TES task has " " or "" set for the Content or URL, it could cause a BadRequest, even if it's inconsequential.

Steps to Reproduce Create a TES task and set the input.Content to empty/whitespace and the input.URL to a real value.

Expected behavior It should be inconsequential.

BMurri commented 1 month ago

I would argue that whitespace in Content is consequential, but empty should be treated the same as null.

MattMcL4475 commented 1 month ago

Leaving as is to allow creation of empty files