-
Notifications
You must be signed in to change notification settings - Fork 3
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
Lots of empty rows #61
Comments
A similar issue has been reported before on user ids containing spaces (#21). But it should have been fixed quite some time ago in Version 0.3.1.
Thanks, it would be great if you could try to identify common properties of the not displayed user ids, especially non-ascii chars, numbers, spaces, etc. I haven't tested the script against v21 yet (but will soon), so this might also be related. Only if it happened by chance, that you have another Nextcloud instance <21 available: |
Could you try to query the API directly with one of the affected user IDs to make sure the issue isn't server-side? |
Thanks for your time @bpcurse. One possible hint; many users have never logged in at least once, could this be a issue? |
Also please note my 1st comment; we have 500 users, we can confirm we have 500 folders at the Also, all usernames are 100% valid email addresses like |
@acsfer Thank you for pointing this out. I somehow didn't get it, sorry ;) In this case there is another direct API query that can aid us in debugging this: If the count is 1200, then the error should be server-side, if it is 500 the culprit is the script (or possibly another server-side error encountered in a later execution step). Simply strip the /username from the API call to receive a userlist instead of data for a specific user |
Ok, i think i found the 'culprit', let me explain. When Nextcloud was first set, all users from LDAP were fetched. After initial deploy, using LDAP, we've applied a filter to only fetch users ending in
The script seems to fetch them all (1200); as they don't "exist" on nextcloud, or are filtered, output is somehow empty for them on the script. Does this seem correct? |
Well it makes it even more mysterious :) If the initial list contains 500 user IDs than the script should only query those 500 IDs individually. As both issues (this one and #64) are on systems using LDAP, this might be as well related. We'll see. I would be interested in the statistics (time needed for the single queries). Could you provide them, too? A debug version is in the works to show us more details from the process. |
Any news on this? :) |
@acsfer Actually, yes :) Thanks for asking. I've been quite busy and was missing spare time for this project in the last months. But "thanks" to certain global circumstances I will probably be able to invest a relevant amount of time between Christmas and the New Year. A debug version and refactored code including a better design will probably be available very early in 2022. So stay tuned ;) |
Thanks for the feedback. No problem, everyone lacks spare time :-D |
Maybe related including possible solution: #65 |
Half the rows displayed contains no data (script detects 1200 users instead of 500, so 700 are just empty)
Happy to help debug this.
The text was updated successfully, but these errors were encountered: