joshp23 / YOURLS-AuthMgrPlus

Seperates user data & manages authorization with role-based access controls (RBAC)
GNU General Public License v3.0
34 stars 19 forks source link

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

Open yusufk opened 4 years ago

yusufk commented 4 years ago

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 commented 4 years ago

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

joshp23 commented 4 years ago

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 commented 4 years ago

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

joshp23 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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

keithdhsb commented 1 year ago

@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.