Inconsistent OAuth callback URL generation between Craft 4 and 5 versions

[juddlyon]

Describe the bug

Video Picker generates different callback URLs between Craft 4 and 5, causing OAuth redirect_uri_mismatch errors when using the same Google OAuth client credentials across sites.

• Craft 4 (v1.x): https://example.com/video-picker/auth/callback/ (with trailing slash)
• Craft 5 (v2.x): https://example.com/video-picker/auth/callback (no trailing slash)

Steps to reproduce

1. Set up Video Picker on Craft 4.x (v1.x plugin)
2. Set up Video Picker on Craft 5.x (v2.x plugin)
3. Use same Google OAuth client credentials
4. Compare generated callback URLs

Craft CMS version

Craft 4.15.0.2 & Craft 5.7.11

Plugin version

1.0.3 & 2.0.3

Multi-site?

Craft 4 (No), Craft 5 (Yes)

Additional context

Both sites have addTrailingSlashesToUrls(true) config and I double-checked site URLs in ENV vars.

[engram-design]

Strange, I'm not seeing that on my end, both Craft 4.15.7 and Craft 5.7.11 have the same URL exactly.

[juddlyon]

You're able to see a trailing slash on the Redirect URI in Craft 5?

We got around it by adding both trailing slash and non at Google Cloud, but it was really subtle to figure out what was broken.

[engram-design]

Nope, both are the same. Super weird!

The only thing I can think of is the usePathInfo config setting, but I imagine both installs are configured the same...

[juddlyon]

Just to make sure I follow for future reference, if I set Craft 5 to ->addTrailingSlashesToUrls(true) the Video Picker Redirect URI will not have it, correct?

[engram-design]

I didn't actually know that was a config setting, but so long as that's consistent in your Craft 4 and 5 project, Video Picker will respect that in either including or excluding the trailing slash. So with that set to true, the URL would be https://craft5.test/video-picker/auth/callback/ (for my install)

[juddlyon]

Thanks @engram-design, always appreciate your help.