better handling of empty storage secrets in aws_util

[adamsachs]

Closes PROD-2839

Description Of Changes

Haven't done any manual testing yet, which should likely be done before we merge.

Code Changes

• [ ] list your code changes here

Steps to Confirm

• [ ] list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

• [ ] All CI Pipelines Succeeded
• Documentation:
  • [ ] documentation complete, PR opened in fidesdocs
  • [ ] documentation issue created in fidesdocs
  • [ ] if there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
• [ ] Issue Requirements are Met
• [ ] Relevant Follow-Up Issues Created
• [ ] Update CHANGELOG.md
• [ ] For API changes, the Postman collection has been updated
• If there are any database migrations:
  • [ ] Ensure that your downrev is up to date with the latest revision on main
  • [ ] Ensure that your downgrade() migration is correct and works
  • [ ] If a downgrade migration is not possible for this change, please call this out in the PR description!

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

| Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **fides-plus-nightly** | ⬜️ Ignored ([Inspect](https://vercel.com/ethyca/fides-plus-nightly/FMsnnv3VHLE363VipzaSVmDaoYWD)) | | | Oct 3, 2024 4:08pm |

[cypress[bot]]

fides    Run #10276

Run Properties:   Passed #10276  •  7312caf9e8 ℹ️: Merge 9367e04cf205bdc534a37caec857952c9285e9ce into c0fc642c0b6abc9e66e32b26a13f...

Project	fides
Branch Review	refs/pull/5347/merge
Run status	Passed #10276
Run duration	00m 39s
Commit	7312caf9e8 ℹ️: Merge 9367e04cf205bdc534a37caec857952c9285e9ce into c0fc642c0b6abc9e66e32b26a13f...
Committer	Adam Sachs
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	0
Skipped	0
Passing	4
View all changes introduced in this branch ↗︎