Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incompatible with https://github.com/8thwall/google-auth-yourls ? #26

Open
yusufk opened this issue Aug 31, 2020 · 7 comments

Comments

@yusufk
Copy link

yusufk commented Aug 31, 2020

Hi

I tried using this plugin together with google-auth-yourls and it seems to be incompatible. Added the email address I used to authenticate into the administrator list and I get:
Require permissions to show admin interface.

@joshp23
Copy link
Owner

joshp23 commented Sep 2, 2020

When using google-auth-yourls, what is the listed username in the admin interface after logging in? Does it display the email address?

@joshp23
Copy link
Owner

joshp23 commented Sep 3, 2020

If you can provide me with temporary Google OAuth client secrets, etc, for your domain, I can set up a testing environment and begin working on compatibility.

@yusufk
Copy link
Author

yusufk commented Nov 11, 2020

It doesn't seem to set a username, as it bypasses the auth flow.

@joshp23
Copy link
Owner

joshp23 commented Mar 12, 2021

The only way to make this work is to retrieve some of whatever data google returns upon login and tie that to a local AMP user. There is nothing listed near the logout button in YOURLS?

@nrasmus
Copy link

nrasmus commented May 4, 2021

We were looking at this plugin, but are also using the Google Auth plugin. No--there is no logout button either, actually. I think OATH is setting a cookie that YOURLS respects as authenticated, but no user data is shared? Not positive.... Would be cool to get these working together though!

@nrasmus
Copy link

nrasmus commented May 5, 2021

This PR for google-auth-URLS promises to fix this--but we haven't gotten that code working with our install yet.

@keithdhsb
Copy link

@nrasmus it does fix it just hasent been commited then all you do is put users full email into one of the sections $amp_role_assignment = array( 'administrator' => array( ''email1@domain.tld', ), 'editor' => array( 'email2@domain.tld', ), 'contributor' => array( ''email3@domain.tld', ),
I know this is old thread and sorry for addind to it.

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

No branches or pull requests

4 participants