-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Error in nextcloud.log after logout #20490
Comments
OS: Debian Sid
|
I have this issue too. I opened a thread about it few days ago. There are more information about my setup etc. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Probably related to: nextcloud/maps#397 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This issue happens for me systematically when refreshing/fetching articles from the Nextcloud News for Android:
System Information
|
In my case the behavior turned out to be a database connection issue. |
How did you find the issue? Maybe it could solve it at other installation aswell. |
I reviewed my stack, re-checking for possiblie issues. The shortfall for me was that when the nextcloud install process ran, at what point appeard to me to be completion of the install process (1st time install for me), the error displayed was not indicative of the actual problem. "session_start(): A session had already been started - ignoring at /opt/nextcloud/lib/private/Session/Internal.php#205" The "nextcloud pre-test for satisfactory resources" (eg.checking for database and php modules) had completed successfully, passing my system as ready for the install. The nextcloud pre-test for install prerequisits DID correctly identify that I had a suitable database installed (as it had failed previously and I remedied that) , however the nextcloud pre-test did NOT identity that the database php/apache connection was not working, php was not connecting to the database. The actual problem was that the php/apache database module was not engaing the database (postgresql in my case). In spite of appearing that the nextcloud install had completed resulting with a session() issue, infact the nextcloud install (post-downloading the nextcloud files) must have not even begun, as there was no database connection working at all. The nextcloud developers and community have done a fantastic job - nextcloud is very impressive! - thanks to all! PS. Now that I have identified the source of my issue, perhaps makes sence to move my post here to #17185 . |
@Rob3487 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
for me the same happens. maybe it is an API related problem. It happens for me for the news and the notes native app. means the web version works fine, but if I start the android apps I get this error. btw. the native nextcloud app works without any error. |
This comment has been minimized.
This comment has been minimized.
That’s because the app does not log-out |
what you mean with logout. normally you call session_start to create a new session or to resume a session based on submitted session id and you do this on every request. Why should a app log-out for every new request? or to rephrase, which kind of bug report should I send to the native apps developers of the news and notes app. Is there any documentation about this changed nextcloud behavior I could forward? |
Try the following. Do not logout. Which is similar to closing the mobile app. |
I debugged it a little bit more. when the error occurs the following happens.
to summarize. I think the function OC\Session\Internal->clear has an error, because it is calling session_regenerate_id followed by a session_start. But session_regenerate_id was already starting the session. |
This will fix errors like "A session had already been started" The reason was that there was a already started session and ->regenerateId() was assigning a new session id to it. So there is no need to call start_session() again. This will fix nextcloud#20490
@HolgerHees thanks, can you make a PR about this for reviewing? |
Hello Everyone, Why is this still open if a fix has been made. I ended up using @HolgerHees Fix when looking at Internal.php and it seems to be working"? Is this a temp fix? and the community as a whole are looking into a more permit fix? Thank you for reading, |
Fixes #20490 Basically restroring the old behavior. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Fixes #20490 Basically restroring the old behavior. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Wondering whether the fix made it to 19.0.1 -- as running that version I still can see them. |
Stiil having the same. |
I probably have the same problem. However, only in Firefox (currently 121) I have opened Chrome in parallel and am also logged into Nextcloud there. There I can access all apps and the settings without a server error (Nextcloud: 27.1.5, PhP: 8.1) |
@osm-frasch Can you open a dedicated issue with your full details? This issue has been closed for several years so it's likely that - even if related - it's an independent issue. That said, if you're getting a internal server error, your |
How to use GitHub
Steps to reproduce
Expected behaviour
No error messages in log
Actual behaviour
The nextcloud log shows an error each time a user logs out. So far, I can see no further effects, but it might be interesting for you to find the cause for this error.
Server configuration
Operating system: Ubuntu 18.04.3
Web server: apache 2.4.29
Database: mariadb 10.1.44
PHP version: 7.2.24
Nextcloud version: 19.0.0.3 (Beta 2)
Updated from an older Nextcloud/ownCloud or fresh install: updated
Where did you install Nextcloud from: nextcloud.com
Signing status:
Signing status
List of activated apps:
App list
Nextcloud configuration:
Config report
Are you using external storage, if yes which one: no
Are you using encryption: no
Are you using an external user-backend, if yes which one: no
Client configuration
Browser: Firefox 75.0
Operating system:
Logs
Web server error log
Web server error log
Nextcloud log (data/nextcloud.log)
Nextcloud log
The text was updated successfully, but these errors were encountered: