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

Nette 2.3#26

Open
tomas-kulhanek wants to merge 3 commits into
Kdyby:masterfrom
tomas-kulhanek:nette-2.3
Open

Nette 2.3#26
tomas-kulhanek wants to merge 3 commits into
Kdyby:masterfrom
tomas-kulhanek:nette-2.3

Conversation

@tomas-kulhanek

Copy link
Copy Markdown

No description provided.

@foxycode

Copy link
Copy Markdown

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

@fprochazka

Copy link
Copy Markdown
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
Copy Markdown

@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

fprochazka commented Mar 29, 2019

Copy link
Copy Markdown
Member

@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

hrach commented Mar 29, 2019

Copy link
Copy Markdown

@fprochazka I suggest archiving these repositories.

@fprochazka

Copy link
Copy Markdown
Member

@hrach that's not a bad idea

@fprochazka

Copy link
Copy Markdown
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