Tusky icon indicating copy to clipboard operation
Tusky copied to clipboard

Default to Login with Browser if Calckey instance

Open ThatOneCalculator opened this issue 2 years ago • 9 comments

In the development branch of Calckey, we added inital support for Mastodon auth and API, and it works in apps like Megalodon, tooot, Elk, etc. It seems the URL tusky uses for authentication causes Calckey to error out though.

Screenshot_20230212-154029.png

ThatOneCalculator avatar Feb 13 '23 00:02 ThatOneCalculator

Sounds like a Calckey issue though?

connyduck avatar Feb 13 '23 07:02 connyduck

Other clients handle it perfectly fine though, maybe see if nodeinfo shows software is "calckey", open the auth window in the browser instead of the embedded WebView?

ThatOneCalculator avatar Feb 13 '23 08:02 ThatOneCalculator

Can you tell me what calckey instance I can use to debug this please? I don't think we will add special handling just for calckey though.

connyduck avatar Feb 13 '23 18:02 connyduck

https://stop.voring.me (recommended) or https://bz.pawdev.me

ThatOneCalculator avatar Feb 13 '23 20:02 ThatOneCalculator

Update: someone in the comments to my post pointed out that you can hit the 3 dots in the top right to log in with browser -- this option works great! I think a simple nodeinfo check to see if software is calckey, then to use that option if true would be enough to provide a smooth option for Calckey users.

Screenshot_20230213-152017.png

ThatOneCalculator avatar Feb 13 '23 23:02 ThatOneCalculator

For whatever reason Calckey login needs Local Storage to work, but if I enable that it fails with another JS error

Screenshot 2023-02-14 at 19 12 43

We should switch back to Custom tab being the default login I guess

connyduck avatar Feb 14 '23 18:02 connyduck

I'm not sure what the service worker error is about, but custom tabs would work.

ThatOneCalculator avatar Feb 14 '23 18:02 ThatOneCalculator

@connyduck any chance we can see an update on this? Should be a relatively simple fix, and Calckey is even more compatible with the Mastodon API now.

ThatOneCalculator avatar Jul 09 '23 22:07 ThatOneCalculator

@ThatOneCalculator Object.fromEntries is a relatively new ECMAScript API, so the error here could be that that version of the browser being used by the operating system doesn't support — it may need a polyfill

https://caniuse.com/?search=object.fromEntries

ThisIsMissEm avatar Apr 03 '24 00:04 ThisIsMissEm