search

solgenomics / breedbase_dockerfile

The Dockerfiles for breeDBase
MIT License
8 stars 8 forks source link

Sending email confirmation to admin instead of user #47

Closed djbutler closed 1 year ago

djbutler commented 1 year ago

I'm trying to send email confirmation to the admin instead of (or in addition to) the user.

I'm doing this by creating a modified version of sgn_local.conf with the following changes:

user_registration_admin_confirmation  1
user_registration_admin_confirmation_email name@example.com

Finally, I add this line to the breedbase service in docker-compose.yml: ./sgn_local.conf:/home/production/cxgn/sgn/sgn_local.conf

It seems like this should enable admin email confirmation, but it doesn't appear to work. Am I missing something?

lukasmueller commented 1 year ago

You also need to open port 25 for the docker (similar to 8080:8080, add an 25:25). Note that the host should not use this port, you may have to deactivate postfix or exim4 on the host.

djbutler commented 1 year ago

I already have a 25:25 clause in my docker compose config file (https://github.com/solgenomics/breedbase_dockerfile/blob/master/docker-compose.yml).

Sorry, I should have been more specific - the confirmation emails are sent out successfully, but they are sent directly to the user-provided email address. I want a way to send confirmation emails to the admins, so that they can confirm that a particular user has permission to create an account. Does that make sense?

An alternative solution would be to filter based on email domain name - but it wasn't obvious to me whether that had been implemented.

lukasmueller commented 1 year ago

Yes, sending the emails to the admin should be possible, but it is a fairly recent feature. Which version of the docker are you running?

cheers Lukas

From: Dan Butler @.> Sent: Monday, June 19, 2023 9:44 PM To: solgenomics/breedbase_dockerfile @.> Cc: Lukas A. Mueller @.>; Comment @.> Subject: Re: [solgenomics/breedbase_dockerfile] Sending email confirmation to admin instead of user (Issue #47)

I already have a 25:25 clause in my docker compose config file (https://github.com/solgenomics/breedbase_dockerfile/blob/master/docker-compose.yml).

Sorry, I should have been more specific - the confirmation emails are sent out successfullu, but they are sent directly to the user-provided email address. I want a way to send confirmation emails to the admins, so that they can confirm that a particular user has permission to create an account. Does that make sense?

An alternative solution would be to filter based on email domain name - but it wasn't obvious to me whether that had been implemented.

— Reply to this email directly, view it on GitHubhttps://github.com/solgenomics/breedbase_dockerfile/issues/47#issuecomment-1598001375, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAAV7F7JBZ4KR5W237WDDL3XMD57DANCNFSM6AAAAAAZJWXEKM. You are receiving this because you commented.Message ID: @.***>

djbutler commented 1 year ago

Thanks, I'm running the docker image with the tag "v0.54".

lukasmueller commented 1 year ago

The current version is v0.75.

Please update and run the patches, in the db/ folder, run_all_patches.pl

thanks and cheers Lukas

From: Dan Butler @.> Sent: Tuesday, June 20, 2023 4:32 PM To: solgenomics/breedbase_dockerfile @.> Cc: Lukas A. Mueller @.>; Comment @.> Subject: Re: [solgenomics/breedbase_dockerfile] Sending email confirmation to admin instead of user (Issue #47)

Thanks, I'm running the docker image with the tag "v0.54".

— Reply to this email directly, view it on GitHubhttps://github.com/solgenomics/breedbase_dockerfile/issues/47#issuecomment-1599456567, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAAV7F5GGA2E4CFIDEDLFVDXMICHFANCNFSM6AAAAAAZJWXEKM. You are receiving this because you commented.Message ID: @.***>

djbutler commented 1 year ago

Thanks I'll give that a try. It looks like docker-compose.yml in this repository is a bit out of date, could be worth updating.

lukasmueller commented 1 year ago

Thanks for the pointer. I updated it to the latest version.

From: Dan Butler @.> Sent: Tuesday, June 20, 2023 7:58 PM To: solgenomics/breedbase_dockerfile @.> Cc: Lukas A. Mueller @.>; Comment @.> Subject: Re: [solgenomics/breedbase_dockerfile] Sending email confirmation to admin instead of user (Issue #47)

Thanks I'll give that a try. It looks like docker-compose.yml in this repository is a bit out of date, could be worth updating.

— Reply to this email directly, view it on GitHubhttps://github.com/solgenomics/breedbase_dockerfile/issues/47#issuecomment-1599750567, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAAV7FYNWLDCLELDBRSGELTXMI2LJANCNFSM6AAAAAAZJWXEKM. You are receiving this because you commented.Message ID: @.***>

djbutler commented 1 year ago

Yep, newer version worked perfectly, thanks very much.