Issues in ProcessPayload.set_id()

[jkeifer]

We definitely have a bug here:

cirrus-geo/src/cirrus/lib2/process_payload.py

Line 181 in a31362b

input_collections = cols if len(cols) != 0 else "none"

. If no collection is specified on the items and collections is not set in the process block then we get a string none and end up with an id like n/o/n/e/...

At the very least we need to wrap that in a tuple or something.

But generally, the whole function could use some thought. What payload fields are we still supporting? process.collections seems really confusing when we have process.upload_options.collections. What is the deal with process.input_collections, is that still supposed to be a thing?