fix: client-side baseUrl is a path

[tills13]

NOTE:

• It's a good idea to open an issue first to discuss potential changes.
• Please make sure that you are NOT opening a PR to fix a potential security vulnerability. Instead, please follow the Security guidelines to disclose the issue to us confidentially.

☕️ Reasoning

Maintains the existing behaviour without using baseUrl which can be either a URL (server-side) or a path (client-side). Since baseUrl is used elsewhere in the function, and the aforementioned behaviour seems to work for the rest of the function body, it seemed easier to just skip trying to make a URL out of the response when what we really wanted was just a query parameter.

🧢 Checklist

• Documentation
• Tests
• Ready to be merged

🎫 Affected issues

Please scout and link issues that might be solved by this PR.

Fixes: #9279

📌 Resources

• Security guidelines
• Contributing guidelines
• Code of conduct
• Contributing to Open Source

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
auth-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 30, 2023 11:59am

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
next-auth-docs	⬜️ Ignored (Inspect)	Visit Preview		Nov 30, 2023 11:59am

[vercel]

@tills13 is attempting to deploy a commit to the authjs Team on Vercel.

A member of the Team first needs to authorize it.

[tills13]

BTW: I just want to add that the dev setup process in this repo is mint. Clone to fix took like 15 minutes. Well done.

[remib18]

Hello,

I've identified that this PR might be linked to an issue with the credentials provider (mentioned in issue #9309). Could someone provide some insight into the rationale behind these changes? I'm interested in understanding more to potentially contribute to a solution.

Edit: After further inspection, it seems that the problem was solved by the latest commit.

[DevYemi]

Hello,

I've identified that this PR might be linked to an issue with the credentials provider (mentioned in issue #9309). Could someone provide some insight into the rationale behind these changes? I'm interested in understanding more to potentially contribute to a solution.

Edit: After further inspection, it seems that the problem was solved by the latest commit.

The problem still persists