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

Logging of mails that could not be delivered. #5015

Closed
1 task done
Nexulo opened this issue May 17, 2024 · 1 comment
Closed
1 task done

Logging of mails that could not be delivered. #5015

Nexulo opened this issue May 17, 2024 · 1 comment

Comments

@Nexulo
Copy link

Nexulo commented May 17, 2024

Describe the feature you'd like

An entry should be added to the logs if an e-mail could not be delivered during registration.

Describe the benefits this would bring to existing BookStack users

In my case, I had mistakenly entered an invalid / non-existent e-mail address when registering. The application then showed a message/toastr after registration that the e-mail could not be sent.

After a few adjustments in EmailConfirmationService.php, I was able to see that I had entered an email address that did not even exist for my domain.

It would be good if this were visible in the logs.

Can the goal of this request already be achieved via other means?

No

Have you searched for an existing open/closed issue?

  • I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?

Under 3 months

Additional context

I searched the issues here and saw that some people also had problems and possibly used a non-existent e-mail address to register.

@ssddanbrown ssddanbrown added this to the Next Feature Release milestone Nov 28, 2024
@ssddanbrown ssddanbrown removed this from the v24.12 milestone Dec 12, 2024
@ssddanbrown
Copy link
Member

Look like we've added extra handling and specific logging for this as part of the changes of d12e8ec, and thread #5195, which was part of v24.10 so I'm therefore going to close this off. Thanks for the original request here @Nexulo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants