chanzuckerberg / idseq-workflows

Portable WDL workflows for IDseq production pipelines
https://idseq.net/
MIT License
31 stars 12 forks source link

Santitize inputs for safe CSVs #116

Closed morsecodist closed 3 years ago

morsecodist commented 3 years ago

At first I wanted to do this to the CSV directly. Unfortunately, I realized we use that CSV downstream for our counts so the identifiers must match. Though it is a bit indirect I think the only way forward is to do this in validate input or not make this CSV available for download at all. Some people may use these characters in their identifiers.

morsecodist commented 3 years ago

Sorry for the unfinishedness I am using the CICD to run tests

morsecodist commented 3 years ago

closing in favor of https://github.com/chanzuckerberg/idseq-workflows/pull/119