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

Hybridauth 2.13 #22

Closed
wants to merge 4 commits into from
Closed

Hybridauth 2.13 #22

wants to merge 4 commits into from

Conversation

GunoH
Copy link

@GunoH GunoH commented May 3, 2020

Upgrade Hybridauth to 2.13 (based on PR 20).
This fixes issue 19 for me.

@GunoH GunoH closed this May 3, 2020
@GunoH
Copy link
Author

GunoH commented May 3, 2020

Having some issues still. I'll test some more and reopen when done.

@GunoH
Copy link
Author

GunoH commented May 5, 2020

Resolved the issue; turns out Hybridauth did not use the same session as Piwigo did. After disabling the Piwigo session handler ($conf['session_save_handler'] = ''; in config.inc.php) things were working fine.

@GunoH GunoH reopened this May 5, 2020
@mistic100
Copy link
Member

mistic100 commented May 5, 2020

Resolved the issue; turns out Hybridauth did not use the same session as Piwigo did. After disabling the Piwigo session handler ($conf['session_save_handler'] = ''; in config.inc.php) things were working fine.

Are you saying this is a prerequisite to make the plugin work ? If so then it's a no go of course, the plugin cannot be so tightly dependant of the config.

Moreover when the default configuration is problematic.

@GunoH
Copy link
Author

GunoH commented May 5, 2020

the plugin cannot be so tightly dependant of the config.
Moreover when the default configuration is problematic.

Yeah, I agree. Cannot be merged the way it is now.
Should find a way to make sure the same session is used by both Piwigo and HybridAuth, but I'm not really sure how.

I'll close this one for now.
If anyone's got any tips on how to proceed, please let me know (though I'm not planning to invest a lot of time in this issue).

@GunoH GunoH closed this May 5, 2020
@kchen
Copy link

kchen commented May 16, 2021

I upgraded to Hybridauth 2.17 and made no other changes to Piwigo or Social Connect, and that appears to resolve the problem for me.

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

Successfully merging this pull request may close these issues.

oAuth with Google Apps fail...
4 participants