feat(kyc-controller): throw on all errors rather than silently log

[jiexi]

Explanation

Previously we had only switched some methods from silent throwing to explicit throws. This PR switches the remaining silent throws. This is being done because the silent throwing pattern causes significant grief and results in awful error handling on the caller side

References

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've communicated my changes to consumers by updating changelogs for packages I've changed
• I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them