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

GNIP - Replace custom auth module with OAuth #1329

Closed
ingenieroariel opened this issue Jan 15, 2014 · 9 comments
Closed

GNIP - Replace custom auth module with OAuth #1329

ingenieroariel opened this issue Jan 15, 2014 · 9 comments
Labels
gnip A GeoNodeImprovementProcess Issue
Milestone

Comments

@ingenieroariel
Copy link
Member

Overview

Taken from Andreas' email:

Currently, GeoServer in GeoNode uses an extension to inherit Django's
security settings.

Now that there is an oauth module available for GeoServer's Spring
security model (http://static.springsource.org/spring-security/oauth/support.html),
and several oauth implementations also exist for Django (e.g.
django-oauth, https://bitbucket.org/david/django-oauth-plus/wiki/Home),
I think it should be possible to get a standards compliant and
extensible way for Django and GeoServer to share GeoNode
authentication/authorization. As a positive side effect, GeoNode could
then also integrate better with other social networking platforms that
support oauth, which would maybe even give us the social networking
features we want for GeoNode for free.

Proposed By:

Assigned to release:

State:

Motivation:

Proposal:

Use cases:

Issues:

Testing:

Alternatives

Feedback

@ingenieroariel ingenieroariel modified the milestone: 2.1 Apr 16, 2014
@capooti
Copy link
Member

capooti commented Apr 16, 2014

+1 for me.

@mwengren
Copy link
Member

+1

@ischneider
Copy link
Member

While I agree a standard approach to authentication would be valuable, due to the way GeoServer authorizes layer access, my guess is that this will perform very poorly under any load if used for authorization.

The reason I had to create the DB client was time-outs resulting from heavy HTTP traffic between the two. The DB client is 2 orders of magnitude faster than the old method and put no burden on the django server.

@jj0hns0n
Copy link
Contributor

Lets discuss security during the sprint. I would love to have external authentication in geonode with social or oauth and know that this works with geoserver in mapstory.

@jj0hns0n jj0hns0n added this to the 2.5.x milestone Jan 13, 2015
@jj0hns0n
Copy link
Contributor

Punting to 2.5

@jj0hns0n jj0hns0n added feature A new feature to be added to the codebase and removed needs-triage labels Jan 13, 2015
@jj0hns0n jj0hns0n modified the milestones: 2.7, 2.5 Aug 21, 2016
@jj0hns0n
Copy link
Contributor

@BerryDaniel lets talk about this now that its documented. May just be able to close? Moving to 2.7 anyway.

@afabiani
Copy link
Member

We are currently working on a technical proposal for this.

The OAuth2 prototype for GeoServer is working and is almost ready.

About the authorization, the idea is to allow GeoNode to act as UserRoleProvider on per-layer request instead of providing the whole list of authorization like it does now.

For more details please take a look here

https://docs.google.com/document/d/1bF1IyFnfOEvShLWcjZ4exImscuOYF31Wjz-8EBQXEmE/edit?usp=sharing

and specifically for GeoNode here

https://docs.google.com/document/d/1fEs6NsCVC3KcB8TCM_P7BmxKLrza2n9fxlySdQVShQ8/edit?usp=sharing

@jj0hns0n
Copy link
Contributor

Ok, leaving this in 2.7 then.

@jj0hns0n
Copy link
Contributor

@afabiani @ingenieroariel going to close this as the new A&A has been merged which implements oAuth. We can open new tickets for this integration separately now.

@afabiani afabiani removed the feature A new feature to be added to the codebase label Aug 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gnip A GeoNodeImprovementProcess Issue
Projects
None yet
Development

No branches or pull requests

6 participants