Open nicovd737 opened 2 months ago
I found some commands by looking in the code:
| polls:db:rebuild
| Rebuild Polls' database including indices |
| polls:index:create
| Create all necessary indices and foreign key constraints |
Could this can solve my issue ? If i run a rebuild operation, it ask me for a backup before running but is it a full nextcloud db backup ? If yes, which tables should i restore in case of trouble ?
Or i am on the wrong way ?
I found some commands by looking in the code:
The commands will not help here.
ok thanks. So any idea of what is the issue or to solve it ? Thanks
Not yet. I need to understand the issue first.
Renaming of users (at least, if only the displayname is changed) should have no effect on access. Only the username (the userId) is relevant.
You can help by descibing your observation more detailed:
Just wanted to keep you from doing things, which will not help and added this quick answer.
@come-nc Maybe you have an idea. In the past there were aleady some problems by searching users, when an LDAP is in play.
To search for users, I actually just call a core function ISearch::search()
.
Hello, Thanks for your reply. The ID was changed also not only the username. Just to be more clear, as an exemple, we had a user named john.who and the user told us that there was an error in the name and it james.who instead. So both the username and the ID were updated, because we are using SAML so we need to have same configuration between AD and Nextcloud otherwise SAML will not work. But that's another subjet :-) All other apps (form, file sharing, meetings,...) works with no problem (after user renaming) for user search except poll. Don't know if it helps.
Thanks for the good explanation. I am wondering, how the id change is tracked inside Nextcloud, so that user properties get updated. At least Polls has not such a function.
But this explains your observation. Polls stores the user by its user id (which is the only and unique identifier of an authenticated user). If a user now enters a poll with a different user id, he can't be recognized, since Polls does not know anything about the changed user id.
To stay in your example, the owner of a poll, an option and a vote or the sharee of a share ist still john.who. james.who is totally another user.
@nicovd737 There is no such thing as userid renaming supported by Nextcloud.
How do you do the renaming? On SAML side only or do you mess with Nextcloud database?
If the userid changes on SAML side I think Nextcloud will map it to a new user.
Hello,
After updating to release 7.2.2, we can invite people again by writing the complet mail (was not possible before). We were force the rename users on database level. We first renamed on AD side but nextcloud did not update ...
So, I don't know much of the AD implementation, but I would say, if you rename the user id on database level, the link is possibly broken. Polls as an app has no chance, to get aware of the changed user id.
I tend to close this issue.
Hello.
Problem for me is that all other nextcloud app did detect the username change. Only polls is not working.
That's a real issue for me.
Thanks for your help.
Nico
Le dim. 29 sept. 2024, 11:30, René Gieling @.***> a écrit :
So, I don't know much of the AD implementation, but I would say, if you rename the user id on database level, the link is possibly broken. Polls as an app has no chance, to get aware of the changed user id.
I tend to close this issue.
— Reply to this email directly, view it on GitHub https://github.com/nextcloud/polls/issues/3693#issuecomment-2381282386, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG3NTYRDDN4R76IX3CEMZ23ZY7CCXAVCNFSM6AAAAABNWEXEH6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRGI4DEMZYGY . You are receiving this because you were mentioned.Message ID: @.***>
I understand your problem. But I can't offer a solution. The search asks the Nextcloud server backend for names, that match the desired search string. In this case it uses a function provided by the server.
If the searched user is not returned, there is nothing what can be done in Polls.
The only chance to get behind this problem is by getting support of the Nextcloud team.
@come-nc Any idea, what the difference could be between Poll's strategy of searching users and this of the other apps?
This is the search call:
It’s the first time I hear about OCP\Collaboration\Collaborators\ISearch
. Maybe the other apps use the IUserManager instead directly to get the users.
But an instance where user ids are messed like that will have tons of trouble, you should not try to make polls work in that situation, it will never perfectly work.
⚠️ This issue respects the following points: ⚠️
What went wrong, what did you observe?
We change some user names following typo errors on nextcloud. Everything is fine and all apps (files, forms, meetings, ...) but with polls app, users can't be found anymore in share options.
What did you expect, how polls should behave instead?
Find users to invit them in polls
What steps does it need to replay this bug?
Installation method
Installed/updated from the appstore (Apps section of your site)
Installation type
Updated from a minor version within same major version (i.e. 4.0.0 to 4.1.1)
Affected polls version
7.2.1
Which browser did you use, when experiencing the bug?
Other browser
No response
Add your browser log here
Additional client environment information
No response
NC version
Nextcloud 28
Other Nextcloud version
No response
PHP engine version
PHP 8.3
Other PHP version
No response
Database engine
MariaDB
Database Engine version or other Database
No response
Which user-backends are you using?
Add your nextcloud server log here
Additional environment informations
No response
Configuration report
List of activated Apps
Nextcloud Signing status
Additional Information
No response