Fix incorrect authentication url

[pnewell]

Fix incorrect authentication url (was using request url but instead should be using the redirectURL that we get back from google)

[mattmassicotte]

I had to make some substantial changes a while ago, and I must have broken this at that point. Thank you for finding it and fixing it. Hopefully it didn't cause you too much trouble along the way.

[martindufort]

Was it always like this ? Authentication URL?

…

On Thu, Apr 3, 2025 at 3:42 PM Matt Massicotte ***@***.***> wrote: I had to make some substantial changes a while ago, and I must have broken this at that point. Thank you for finding it and fixing it. Hopefully it didn't cause you too much trouble along the way. — Reply to this email directly, view it on GitHub <#30 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG4YD4OZZ5IFQNWWPJZ3V3D2XWFI7AVCNFSM6AAAAAB2NBC5BSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZWG42TSMRYGM> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***> [image: mattmassicotte]*mattmassicotte* left a comment (ATProtoKit/OAuthenticator#30) <#30 (comment)> I had to make some substantial changes a while ago, and I must have broken this at that point. Thank you for finding it and fixing it. Hopefully it didn't cause you too much trouble along the way. — Reply to this email directly, view it on GitHub <#30 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG4YD4OZZ5IFQNWWPJZ3V3D2XWFI7AVCNFSM6AAAAAB2NBC5BSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZWG42TSMRYGM> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[mattmassicotte]

No, I don't think so. I had to introduce some new concepts that added some naming ambiguity. I tried my best to keep these correct, but I think I missed a few spots.

[martindufort]

Just looked at my version and yes there is a lot of changes... That's my loginProvider()

	static func loginProvider() -> TokenHandling.LoginProvider {
		return { url, appCredentials, tokenURL, urlLoader in
			let request = try authenticationRequest(url: url, appCredentials: appCredentials)

			let (data, _) = try await urlLoader(request)
        ...

And you did a lot of changes to support SwiftConcurrency.
Need to setup time to upgrade but only after iPhone launch 🥵🥵