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

Support separating admin and contact email addresses #1647

Open
goetzk opened this issue Nov 5, 2024 · 1 comment
Open

Support separating admin and contact email addresses #1647

goetzk opened this issue Nov 5, 2024 · 1 comment

Comments

@goetzk
Copy link
Contributor

goetzk commented Nov 5, 2024

Increasingly we're finding that the admin email address isn't appropriate for all situations and it would be good to separate 'system admin' and 'from address' in more places.

An example of this is where we manage the CA deployment but the site is managed by an experienced person at the customer site.

I've had a quick look through providence looking for references to __CA_ADMIN_EMAIL__ and ca_admin_email and tried to determine who would be responsible.

  • app/lib/Plugins/TaskQueueHandlers/dataExport.php - data exports are a customer thing
  • app/lib/Utils/CLIUtils/Maintenance.php - fixity reports are a hosting thing
  • app/lib/Utils/CLIUtils/Test.php - test emails are a hosting thing
  • app/lib/Utils/CLIUtils/Localization.php - probably a hosting thing
  • app/lib/BatchProcessor.php - batch processes are probably a customer thing
  • app/models/ca_users.php - account setup probably a hosting thing
  • app/controllers/administrate/access/UsersController.php - account steps hosting, comment notifications customer thing
  • app/conf/oai_provider.conf - is configurable, but i imagine it would default to customer setting.

I suggest a new setting __CA_CONTACT_EMAIL__ or similar to separate the two tasks. I'm aware this introduces the possibility that admin and contact emails would be sent to different mail servers and as such might need separate mail configuration - I haven't dug in to that part of the problem though.

@goetzk
Copy link
Contributor Author

goetzk commented Nov 6, 2024

Oh, and I should say I didn't look at pawtucket at all. I'm aware of other places admin email displays (ban hammer error page for example), the list above is more of a starter than comprehensive list.

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

1 participant