feat(adapter): take away error handling from adapters

[balazsorban44]

Reasoning 💡

What changes are being made? What feature/bug is being fixed here?

Checklist 🧢

Feel free cross items ( like this ~[] item~ ) if they're irrelevant to your changes.

To check an item, place an x in the box like so: - [x] Documentation.

• Documentation
• Tests
• Ready to be merged
• add debugging
  

Affected issues 🎟

#1832

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/nextauthjs/next-auth/3wcqomtEk3nVnTTJhRoKDky8WLP8
✅ Preview: https://next-auth-git-feat-move-adapter-responsibility-nextauthjs.vercel.app

[balazsorban44]

should log out args with debug, and an adapter prefix(?)

@kripod, currently logger.debug is rewritten in adapters to include a prefix for code. Do you think we could return a name property next to getAdapter for this purpose?

any thoughts?

[kripod]

We may specify an attribute like { displayName: string } as the minimum required config param when instantiating adapters. By doing so, we would also solve the typing issue of Adapter(requiredParam, …).

[balazsorban44]

After my recent commit, it will be possible for adapters to return a displayName property in getAdapter. Unfortunately we don't have the adapter configuration when we invoke the .getAdapter() method.

[balazsorban44]

Currently waiting for @kripod and his results after testing these changes in production. We expect fewer unhandled errors.