Skip to content
This repository has been archived by the owner on Mar 29, 2019. It is now read-only.

Nette 2.3 #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Nette 2.3 #26

wants to merge 3 commits into from

Conversation

tomas-kulhanek
Copy link

No description provided.

@foxycode
Copy link

@fprochazka I'd like to continue on this lib. Will you merge my PR's or will you give me maintainer?

@fprochazka
Copy link
Member

I strongly suggest migrating to this: https://github.com/thephpleague?utf8=%E2%9C%93&q=oauth&type=&language=

Maintaining separate libraries for separate providers is unsustainable and doesn't make much sense.

Since a quick google didn't show me any nette integrations for league/oauth2-client, I'd be happy to create a repository for you to create it.

@foxycode
Copy link

@fprochazka You mean creating new repository which will use thephpleague/oauth2-google ? How about rewrite this one, which people know? It was my first gues when I needed Google integration. I don't have problem maintaining it. I will need it in few production apps.

BTW: Why not use google/apiclient:^2.0 ?

@fprochazka
Copy link
Member

fprochazka commented Mar 29, 2019

@foxycode the oauth2-client from league is very generic, where you can plug in any of the providers and you should not have to modify your application much. IMHO the correct way is to provide integration with the generic implementation, then it should be easy to just use whatever provider. Which means this repository will still loose meaning.

Google api client covers a lot of google API's, it does not make sense to install client for all of their services, if all you really want is auth.

It didn't occur to me at the beginning, but maintaining separate libs for each providers is hell. Oauth2 is a standartized protocol and there are only slight variances from the providers. Kdyby/Facebook, Google and Github should have never existed.

@hrach
Copy link

hrach commented Mar 29, 2019

@fprochazka I suggest archiving these repositories.

@fprochazka
Copy link
Member

@hrach that's not a bad idea

@fprochazka
Copy link
Member

@foxycode if you want to discuss this further, I'd be happy to do so in #kdyby channel on Pehapkari slack. Also, archiving the repository is easily reversible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants