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

Nextcloud 24 support #183

Open
susnux opened this issue May 25, 2022 · 11 comments
Open

Nextcloud 24 support #183

susnux opened this issue May 25, 2022 · 11 comments

Comments

@susnux
Copy link

susnux commented May 25, 2022

Looks like nextcloud 24 does not have any critical API changes, and this plugin looks like it works with PHP8.1 (only requirement change in NC24).
Has anyone tested it on NC24? And will NC25 be officially supported?

@millquarterlabs
Copy link

Works partially, login works for most users however some users did apparently not get imported fully or something. The "User" view shows the correct number of users however it does not display all. The "oc_accounts" db contains only 40 users.

@tobox
Copy link

tobox commented Jun 22, 2022

Any news on this?

@Thom1b
Copy link

Thom1b commented Jul 20, 2022

Hi, how can we help to make user_sql available for NC-24?

@jvllmr
Copy link

jvllmr commented Aug 7, 2022

We are using this plugin at the company I am working at with NC24 for quite a while now and we did not run into issues (still with php7 afaik)

@Trooper123
Copy link

Trooper123 commented Aug 11, 2022

Does anyone know when nc24 will be supported?
Are there other systems where the plugin runs under nc24?( please qoute)

@d-jsb
Copy link

d-jsb commented Aug 18, 2022

Having NC 24.0.4 I see this in the log
Allowed memory size of 536870912 bytes exhausted (tried to allocate 16384 bytes) at /var/[...]/3rdparty/doctrine/dbal/src/Driver/PDO/Statement.php#92
First I had only 256M (php limit) but I increased it to recommended 512M. Still not enough? It used to work fine with ~480 Users in DB
Showing groups with small number (<10) of members tend to work better but not all of them show user list.

@d-jsb
Copy link

d-jsb commented Aug 18, 2022

Allowed memory size of 536870912 bytes exhausted (tried to allocate 4096 bytes) at /www/.../apps/user_sql/lib/Query/DataQuery.php#258

@Bakaras
Copy link

Bakaras commented Aug 25, 2022

Version 24 is mentioned in the fork https://github.com/ulukyn/user_sql
Maybe it is the solution?

@poVoq
Copy link

poVoq commented Sep 11, 2022

I am also having issues with this, and it is unclear if that is simply caused by a mis-configuration or lacking NC24 support.

@d-jsb
Copy link

d-jsb commented Sep 11, 2022

Latest NC 24 upgrade today did not improve the situation. As the config was not changed, I guess it is missing support in user_sql.
Unfortunately I did not code in PHP for years, so I have no idea how to fix this.

@d-jsb
Copy link

d-jsb commented Sep 27, 2022

I found I maybe solved my problem by disabling email sync in the user_sql options. Will observe and report in case of new errors.
According to the debug log the EmailSync#doAction appears to run into a recursion, the following log lines repeat over and over again - I did not investigate the details of the DB rows triggering the problem

`{"reqId":"xxx","level":0,"time":"2022-09-27T19:41:24+00:00","remoteAddr":"7xxx1","user":"1","app":"user_sql","method":"GET","url":"/ocs/v2.php/cloud/groups/21/users/details?offset=0&limit=25&search=","message":"Entering userExists(418)","userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:104.0) Gecko/20100101 Firefox/104.0","version":"24.0.5.1","data":{"app":"user_sql"}}
{"reqId":"xxx","level":0,"time":"2022-09-27T19:41:24+00:00","remoteAddr":"7xxx1","user":"1","app":"user_sql","method":"GET","url":"/ocs/v2.php/cloud/groups/21/users/details?offset=0&limit=25&search=","message":"Executing query: SELECT u.id AS uid, u.username AS username, u.name AS name, u.email AS email, null AS quota, null AS home, true AS active, false AS avatar, null AS salt FROM c9m37_users u WHERE u.id = :uid AND NOT u.block, 418","userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:104.0) Gecko/20100101 Firefox/104.0","version":"24.0.5.1","data":{"app":"user_sql"}}
{"reqId":"xxx","level":0,"time":"2022-09-27T19:41:24+00:00","remoteAddr":"7xxx1","user":"1","app":"user_sql","method":"GET","url":"/ocs/v2.php/cloud/groups/21/users/details?offset=0&limit=25&search=","message":"Entering EmailSync#doAction(418)","userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:104.0) Gecko/20100101 Firefox/104.0","version":"24.0.5.1","data":{"app":"user_sql"}}

{"reqId":"xxx","level":0,"time":"2022-09-27T19:41:24+00:00","remoteAddr":"7xxx1","user":"1","app":"user_sql","method":"GET","url":"/ocs/v2.php/cloud/groups/21/users/details?offset=0&limit=25&search=","message":"Entering userExists(418)","userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:104.0) Gecko/20100101 Firefox/104.0","version":"24.0.5.1","data":{"app":"user_sql"}}
{"reqId":"xxx","level":0,"time":"2022-09-27T19:41:24+00:00","remoteAddr":"7xxx1","user":"1","app":"user_sql","method":"GET","url":"/ocs/v2.php/cloud/groups/21/users/details?offset=0&limit=25&search=","message":"Executing query: SELECT u.id AS uid, u.username AS username, u.name AS name, u.email AS email, null AS quota, null AS home, true AS active, false AS avatar, null AS salt FROM c9m37_users u WHERE u.id = :uid AND NOT u.block, 418","userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:104.0) Gecko/20100101 Firefox/104.0","version":"24.0.5.1","data":{"app":"user_sql"}}
{"reqId":"xxx","level":0,"time":"2022-09-27T19:41:24+00:00","remoteAddr":"7xxx1","user":"1","app":"user_sql","method":"GET","url":"/ocs/v2.php/cloud/groups/21/users/details?offset=0&limit=25&search=","message":"Entering EmailSync#doAction(418)","userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:104.0) Gecko/20100101 Firefox/104.0","version":"24.0.5.1","data":{"app":"user_sql"}}
`

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

9 participants