Skip to content

Changed default role for calendar access to show only date and time.#7152

Open
admins-little-helper wants to merge 1 commit intomailcow:stagingfrom
admins-little-helper:fix/sogo-default-calendar-acl
Open

Changed default role for calendar access to show only date and time.#7152
admins-little-helper wants to merge 1 commit intomailcow:stagingfrom
admins-little-helper:fix/sogo-default-calendar-acl

Conversation

@admins-little-helper
Copy link
Copy Markdown

Contribution Guidelines

What does this PR include?

Short Description

Currently the default calendar shareing permissions in SOGo for events marked as "public" in a personal calendar are set to "ViewAll" for all users in the same domain.
In addition by default new events are created as "public". That results in all users within a domain/org can read events of all other users by default.

To improve security I think the default should be changed to allow only to see date/time of an event ("free/busy"). Every user then still can set sharing permissions individually as needed.

Current default calendar sharing permission:
SOGo_Kalenderfreigabe1a_EN

Proposed change:
SOGo_Kalenderfreigabe2a_EN

Affected Containers

  • sogo-mailcow

Did you run tests?

What did you tested?

Edited file data/conf/sogo/sogo.conf and changed

SOGoCalendarDefaultRoles = (
  PublicViewer,
  ConfidentialDAndTViewer,
  PrivateDAndTViewer
);

to this:

SOGoCalendarDefaultRoles = (
  PublicDAndTViewer,
  ConfidentialDAndTViewer,
  PrivateDAndTViewer
);

Restarted containers sogo-mailcow and memcached-mailcow:

docker compose restart sogo-mailcow memcached-mailcow

What were the final results? (Awaited, got)

Works as intendet: the default calendar sharing permissions for new and existing users are set to show date & time only ("free/busy") for events marked as public. Users can change shareing permisisons as they like. In case a user already has changed sharing permissions before, those individual permissions are not changed.

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

Successfully merging this pull request may close these issues.

1 participant