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

Add possibility to allow/disallow main email, email alias and email forward form portal #1997

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Josue-T
Copy link
Contributor

@Josue-T Josue-T commented Nov 9, 2024

PR related to: YunoHost/yunohost-portal#21

The problem

In Yunohost portal, user is always allowed to edit the mail alias and mail forward rule.

In some instance it could be a security issue to allow to edit by example the mail alias.

Solution

Add 3 new Yunohost settings to define if the user is allowed to edit

  • the main email
  • the mail alias
  • the mail forward

PR Status

Tested locally it work

How to test

  • Checkout the branch on yunohost and yunhost-portal
  • Allow the 3 rules in "Tools" > "Yunohost settings" > "Portal" from admin
  • Test that from the yunohost portal, the user can edit everything.
  • Disallow the 3 rules in "Tools" > "Yunohost settings" > "Portal" from admin
  • Test that from the yunohost portal, the user can't edit everything.

@Josue-T Josue-T requested review from alexAubin and removed request for alexAubin November 28, 2024 06:53
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