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

Meeting creation: permission problem #974

Closed
Elblinator opened this issue Jul 26, 2024 · 3 comments
Closed

Meeting creation: permission problem #974

Elblinator opened this issue Jul 26, 2024 · 3 comments
Labels
bug needs info waiting Waiting for some other PR/feature; more details in comments
Milestone

Comments

@Elblinator
Copy link
Member

Description:
When creating new meetings, it is possible for all committee admins to access the entire pool of ALL active meetings and use them as a basis for a new meeting. This is incorrect behavior, as it allows access to meeting for which a committee admin previously had no access.

Example image - committee admin have access to all active meetings
image

Reproduction:

  1. create two committees (A and B)
  2. create in both committees a meeting which is active
  3. create a new user XYZ and add to committee A as committee admin
  4. Log in as user XYZ and open committee A
  5. create a new meeting. in the "duplicate from" field the meeting from committee B is visible

What should happen?
When creating new meetings by duplication, it should only be possible duplicate from meetings from the same committee or from meeting templates.
Meetings for which the committee admin does not have access authorization, i.e. which are not part of the committee or a template, should not be displayed in the list of possible meetings for duplication.

@Elblinator Elblinator added the bug label Jul 26, 2024
@Elblinator Elblinator added this to the 4.2 milestone Jul 26, 2024
@ostcar
Copy link
Member

ostcar commented Jul 26, 2024

I think, this has to be fixed in the backend. The autoupdate does not clone

@ostcar
Copy link
Member

ostcar commented Jul 27, 2024

If it is only the display, then it has to be fixed in the client. If you want to change a restrictor, I need more information, what field should be restricted. Currently, all meeting names can be seen by everyone.

@Elblinator Elblinator added the waiting Waiting for some other PR/feature; more details in comments label Jul 30, 2024
@MSoeb
Copy link

MSoeb commented Aug 19, 2024

The problem was solved during developement of OS version 4.1.14. The 'Duplicate from..'-field now only shows templates and no other meetings for ALL admins.

@MSoeb MSoeb closed this as completed Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs info waiting Waiting for some other PR/feature; more details in comments
Projects
None yet
Development

No branches or pull requests

3 participants