ember-simple-auth icon indicating copy to clipboard operation
ember-simple-auth copied to clipboard

Is Torii addon abandonned ?

Open BnitoBzh opened this issue 3 years ago • 10 comments

The Torii addon has 72 open issues and 38 pull requests. It has not been updated for over 3 years.

At this time, Torii is required in order to use 3rd party authentications .. But with Ember 3.28 and more, the Torii addon starts to be deprecated in a lot of things

What will the deal be for the future? For now ember-simple-auth needs Torii. So users need to fork Torii to update it? Will the ember-simple-auth maintainers do something integrated?

BnitoBzh avatar Jan 03 '22 10:01 BnitoBzh

The Torii authenticator was recently deprecated in #2352 and will be removed from ESA in a future breaking release. It is possible to built something that replicates what Torii in a custom authenticator inside ESA. There is no roadmap for this as of now.

pichfl avatar Jan 03 '22 16:01 pichfl

Wow ! Ok .. ESA is the most used addon by the Ember community for authentication/authorization implementation and its next breaking release is to remove its social connection feature without a workaround in its roadmap ? I think propose a full built-in solution, an abstract layer or a real example (without Torii) should be do.

BnitoBzh avatar Jan 06 '22 08:01 BnitoBzh

IMHO this is the right decision. Torii is (and always was) unrelated to ESA, and the integration was never nice. Not it seems dead, so better don't hold ESA with it. It is actually pretty straightforward to just use ESA and a custom authenticator.

luxzeitlos avatar Jan 11 '22 03:01 luxzeitlos

The Torii authenticator was recently deprecated in #2352 and will be removed from ESA in a future breaking release. It is possible to built something that replicates what Torii in a custom authenticator inside ESA. There is no roadmap for this as of now.

@pichfl I've wrote down a custom Authenticator to replace Torii (and a service to manage the popup). Do you think is better opening a PR on this repo with that code or creating a separate ember addon for handling that type of authentication flow?

davideferre avatar Feb 28 '22 22:02 davideferre

The Torii authenticator was recently deprecated in #2352 and will be removed from ESA in a future breaking release. It is possible to built something that replicates what Torii in a custom authenticator inside ESA. There is no roadmap for this as of now.

@pichfl I've wrote down a custom Authenticator to replace Torii (and a service to manage the popup). Do you think is better opening a PR on this repo with that code or creating a separate ember addon for handling that type of authentication flow?

@davideferre I am looking to write the same thing and using yours as a reference would be a great help!

LumenMicho avatar Jul 12 '22 16:07 LumenMicho

@pichfl I've wrote down a custom Authenticator to replace Torii (and a service to manage the popup). Do you think is better opening a PR on this repo with that code or creating a separate ember addon for handling that type of authentication flow?

@davideferre I did some work in #2367 but wasn't able to allocate the necessary time yet, maybe we can join efforts here. A PR is always welcome.

pichfl avatar Jul 13 '22 07:07 pichfl

I can share a PR later today, thanks!

davideferre avatar Jul 14 '22 09:07 davideferre

@pichfl @LumenMicho I'm not ready for a PR yet but here you can see a work-in-progress version of what I did some months ago to replace Torii in some of my private apps: https://github.com/davideferre/ember-simple-auth/blob/popup_authenticator/packages/ember-simple-auth/addon/authenticators/oauth2-popup.js Note that at the moment it uses two additional services but probably is not a good thing to add 2 services to ESA just for this authenticator.

davideferre avatar Jul 14 '22 21:07 davideferre

Version 1.0.0-beta.1 of torii has been published.

Source code is located at https://github.com/adopted-ember-addons/torii

Gaurav0 avatar Jan 30 '23 14:01 Gaurav0

ESA is raising a deprecation for using torii, this should be updated

khaled-s avatar Dec 21 '23 13:12 khaled-s