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

Improve pipeline documentation #361

Closed
shoooe opened this issue Aug 20, 2014 · 3 comments
Closed

Improve pipeline documentation #361

shoooe opened this issue Aug 20, 2014 · 3 comments

Comments

@shoooe
Copy link

shoooe commented Aug 20, 2014

The documentation regarding the pipeline is basic to say the least. I think that each step of at least the default pipeline should be described:

    'social.pipeline.social_auth.social_details',
    'social.pipeline.social_auth.social_uid',
    'social.pipeline.social_auth.auth_allowed',
    'social.pipeline.social_auth.social_user',
    'social.pipeline.user.get_username',
    'social.pipeline.user.create_user',
    'social.pipeline.social_auth.associate_user',
    'social.pipeline.social_auth.load_extra_data',
    'social.pipeline.user.user_details'

What does 'social.pipeline.social_auth.social_details' do, for example? Which steps does 'social.pipeline.social_auth.social_user' depend on? And so on.

The code is not documented, so it's hard to tell what parts can be customized and where.

@shoooe shoooe changed the title Improve Improve pipeline documentation Aug 20, 2014
@wadewilliams
Copy link

fwiw,

# Get the information we can about the user.
'social.pipeline.social_auth.social_details',

# Get the social uid from whichever service we're authing thru
'social.pipeline.social_auth.social_uid',

# Some sort of authorization that has nothing to do with whether or not the user granted permissions or not
'social.pipeline.social_auth.auth_allowed',

# Checking the usersocialauth table for pre-existing connections
'social.pipeline.social_auth.social_user',

# Make up a username for this person
'social.pipeline.user.get_username',

# 'social.pipeline.mail.mail_validation',

# Checking against user table for email if we don't have them yet
'social.pipeline.social_auth.associate_by_email',

# Create a user if we haven't found one yet
'social.pipeline.user.create_user',

# Create the socialauth connection record for this user.
'social.pipeline.social_auth.associate_user',


'social.pipeline.social_auth.load_extra_data',
'authentication.services.authentication.socialSendActivationEmail',

# Update the user record with any changed info from social.
'social.pipeline.user.user_details',

That might not be 100% correct but its atleast my understanding of it at this point.

@omab omab closed this as completed Sep 3, 2014
@shoooe
Copy link
Author

shoooe commented Sep 6, 2014

Exactly what I was looking for, thanks @wadewilliams.

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

3 participants