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

Feature Request - User management #67

Closed
mbsouth opened this issue Jun 2, 2023 · 9 comments
Closed

Feature Request - User management #67

mbsouth opened this issue Jun 2, 2023 · 9 comments

Comments

@mbsouth
Copy link

mbsouth commented Jun 2, 2023

I love this tool, great work!

One feature I'm missing is managing users and mapping domains to those users.
This would allow users to log in and view their domain reports.

Another feature would be if users had a certain number of domains available and could specify them themselves (users would then have to take care of DNS records themselves)

@liuch
Copy link
Owner

liuch commented Jun 4, 2023

I planned to add support for user management, but as a restriction on access to existing features without mapping domains to users. The feature you suggested is more than just a feature of a personal mail server tool. I'll think it over.

@liuch liuch mentioned this issue Jul 2, 2023
@williamdes
Copy link
Contributor

Repost from #87

I do not want to host many instances of DmarcSrg. I need to have a way to give a credential to users of domain/domains XYZ so they do not have access to the other ones.

Can we make:

  • a multi password config that each of them has a filter for the allowed domains
  • a user/password table with domains by user
  • a user/password config that each of them has a filter for the allowed domains

What solution would be best ?

@williamdes
Copy link
Contributor

williamdes commented Jul 2, 2023

@liuch said in #87

I think it would be better to have tables users and user_domains or something. The admin user is configured in the config file, other users is in tables. What do you think about this?

I think user domains table would be good
would the admin config disappear?

@liuch
Copy link
Owner

liuch commented Jul 2, 2023

would the admin config disappear?

Would not. The main reason is that the administrator can access the web interface when the database structure is not initialized.

@SteelPC
Copy link

SteelPC commented Jul 21, 2023

Repost from #87

I do not want to host many instances of DmarcSrg. I need to have a way to give a credential to users of domain/domains XYZ so they do not have access to the other ones.

Can we make:

* a multi password config that each of them has a filter for the allowed domains

* a user/password table with domains by user

* a user/password config that each of them has a filter for the allowed domains

What solution would be best ?

I think users should not be allowed to add domain themselves or else they could gain access to unwanted information. Only admin should allow a domain to a user.

@liuch
Copy link
Owner

liuch commented Jul 21, 2023

Yes. I think so. Maybe I'll add a special role for this with some checks. By the way, I have already started writing code to implement user management.

@liuch
Copy link
Owner

liuch commented Aug 13, 2023

This commit 6e8674b contains initial implementation of user management.

  • Each user has its own settings.
  • The user's access level affects the functionality available to the user.
  • Mapping users to domains has not been implemented yet.

I am going to gradually complete the functionality.

added: See config/conf.sample.php ($admin section) for some details.

@liuch
Copy link
Owner

liuch commented Apr 25, 2024

This commit is a continuation of the user management implementation. Brief highlights:

  • Admin has access to all the domains with no limitation;
  • Any domain can be assigned to one or more users (only admin can do this);
  • Users with manager access level can add and remove their domains. If a domain already exists, it will be assigned to this user. Instead of deleting, domains are unassigned. Admin can finally remove them.
  • utils/summary_report.php now has a parameter user that allows you to get summary reports for all domains of a specified user.
  • Other small things.

I'll post the documentation a little later.

@liuch
Copy link
Owner

liuch commented May 1, 2024

I'll post the documentation a little later.

Done.

@liuch liuch closed this as completed May 1, 2024
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

No branches or pull requests

4 participants