(PXP-6580): fix data submission by only writing acl and authz iff both are empty

[johnfrancismccann]

Bug Fixes

• Only write acl and authz fields of corresponding Indexd record during data submission iff both fields are empty to begin with. Prevents authz field from being unintentionally populated in commons not using authz.

Breaking Changes

• Note that this PR will cause Sheepdog behavior to change slightly during a data submission flow in which the acl and/or authz field are previously populated in the corresponding Indexd record and the Indexd uploader field is not null and the Sheepdog submitter does not match the Indexd uploader. While the existing behavior will cause data submission to fail and report an error message to the user, this PR’s changes will mean that data submission can still succeed in this case of populated acl/authz and uploader mismatch. I believe this change in behavior is not problematic in that it would be unexpected in any commons for uploader and (acl and/or authz) to both be populated.

[PlanXCyborg]

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

[Avantol13]

can you add your "note" comment into the "Breaking Changes" section of the PR just so that it gets picked up / read by people reading the release notes when we create a new version?