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.

Sign up for GitHub

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

Jump to bottom

Nextcloud 21 Support #238

Closed
jaddie opened this issue Mar 10, 2021 · 4 comments
Closed

Nextcloud 21 Support #238

jaddie opened this issue Mar 10, 2021 · 4 comments

Comments

@jaddie
Copy link

jaddie commented Mar 10, 2021

Hi,

I am just wondering if this supports Nextcloud & PHP 7.4 fully as I am having some errors in the log related to PHP foreach as well as the login not seeming to work even though the app is all setup and such for Facebook Login.

I am a bit confused what I am possibly doing wrong so wanted to ask first before I dig any further.

Cheers
@zorn-v
Copy link
Owner

zorn-v commented Mar 12, 2021

Which errors do you have in log ?

@martinkoutecky
Copy link

I don't know if this is the same issue, but could be. I only set up social login after upgrade to NC21, so I don't know if what broke it was the upgrade or something else. I'm trying to authenticate against mailcow (custom OAuth2). The login correctly takes me to mailcow, I log in there, confirm I want to give permissions, and then nextcloud gives the following error:

Signed API request has returned an error. HTTP error 404. Raw Provider API response: <html> <head><title>404 Not Found</title></head> <body> <center><h1>404 Not Found</h1></center> <hr><center>nginx</center> </body> </html> .

In the NC log I get something like this

session_start(): Ignoring session_start() because a session is already active at /path/to/NC/lib/private/Session/Internal.php#207

... which looks like nextcloud/server#25807 ... so I'm not sure it's related to the social login not working?

I can provide more logs if you can be specific in what would be helpful. Thanks!

@zorn-v
Copy link
Owner

zorn-v commented Mar 20, 2021

Any other confirmations of NC 21 users that something does not work ?

@martinkoutecky
Copy link

@zorn-v sorry -- I found out the error was caused by some typos in the configuration, so I've got it working and the issue was not related to NC21.

@zorn-v zorn-v closed this as completed Mar 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jaddie @martinkoutecky @zorn-v