-
Notifications
You must be signed in to change notification settings - Fork 73
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
When nextcloud user are renamed, polls don't see renamed users anymore #3693
Comments
I found some commands by looking in the code: | Could this can solve my issue ? Or i am on the wrong way ? |
The commands will not help here. |
ok 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. To search for users, I actually just call a core function polls/lib/Service/SystemService.php Lines 93 to 95 in 2d7c6c7
|
Hello, |
Thanks for the good explanation. 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). |
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
<#3693 (comment)>,
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: polls/lib/Service/SystemService.php Lines 94 to 95 in e35f823
|
It’s the first time I hear about 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. |
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
The text was updated successfully, but these errors were encountered: