John Chilton

Pushing the milestone, but this is a bug fix and we could merge whenever I think. Want to rebase and were the last of Marius' comments addressed?

It did indeed fail. ``` def _verify_outputs(testdef, history, jobs, tool_id, data_list, data_collection_list, galaxy_interactor, quiet=False): assert len(jobs) == 1, "Test framework logic error, somehow tool test resulted in more than one...

xref https://github.com/galaxyproject/galaxy/pull/7894

@natefoo I basically agree with everything @erasche said about it not being actionable really for now and that researchers cannot really do much with data except enjoy that it is...

@guerler About the changes to the security module - I wish they were a bit different. As outlined here: Can you handle the encoding a bit differently? There are a...

> On Node.js, notebook.js runs all "text/html"-type outputs through DOMPurify. The browser-based version, however, does not ship with this library; to enable the default behavior, you must -include or require...

Awesome - amazing and super cool then! What a great approach.

-0 - just use Pulsar rather than making jobs stuff for more complicated on this end. It also doesn't tie into objectstores - I feel like if these files exist...

> should this script stay here or is it better to integrate into (e.g.) ephemeris ... ? This script since it is here for demonstration purposes - I'm not sure...

Actually, there are linting problems - will you fix those? > ./scripts/api/library_upload_dir.py:25:56: W291 trailing whitespace