remove useless (i think?) try-catch

[Rich-Harris]

i've no idea what this err.data stuff was about but it doesn't seem very useful — we should probably just let sveltekit's normal error handling deal with it

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
omnisite	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 20, 2024 1:32pm

[dummdidumm]

The slight difference would be that previously it returned whatever was inside the data property. I have no idea if that is in any way useful/necessary, so I'm inclined to agree that we can just remove the try-catch.

[Conduitry]

Is the contents of the data property going to be some oauth2 error? Is that going to be getting logged somewhere else, if it's no longer going to be returned in the HTTP response?

[Rich-Harris]

Honestly no idea what data is, which by itself feels like a good reason to not just lob it at the client. I would hope that the error message (which would get logged) contains enough information to start diagnosing