mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-06 09:09:38 +03:00
[Bug Report]: HELO not correctly configured when sending emails through SMTP #2494
Closed
opened 2026-02-05 04:20:14 +03:00 by OVERLORD
·
4 comments
No Branch/Tag Specified
development
further_theme_development
l10n_development
release
llm_only
vectors
v25-11
docker_env
drawio_rendering
user_permissions
ldap_host_failover
svg_image
prosemirror
captcha_example
fix/video-export
v25.12.3
v25.12.2
v25.12.1
v25.12
v25.11.6
v25.11.5
v25.11.4
v24.11.4
v25.11.3
v25.11.2
v25.11.1
v25.11
v25.07.3
v25.07.2
v25.07.1
v25.07
v25.05.2
v25.05.1
v25.05
v25.02.5
v25.02.4
v25.02.3
v25.02.2
v25.02.1
v25.02
v24.12.1
v24.12
v24.10.3
v24.10.2
v24.10.1
v24.10
v24.05.4
v24.05.3
v24.05.2
v24.05.1
v24.05
v24.02.3
v24.02.2
v24.02.1
v24.02
v23.12.3
v23.12.2
v23.12.1
v23.12
v23.10.4
v23.10.3
v23.10.2
v23.10.1
v23.10
v23.08.3
v23.08.2
v23.08.1
v23.08
v23.06.2
v23.06.1
v23.06
v23.05.2
v23.05.1
v23.05
v23.02.3
v23.02.2
v23.02.1
v23.02
v23.01.1
v23.01
v22.11.1
v22.11
v22.10.2
v22.10.1
v22.10
v22.09.1
v22.09
v22.07.3
v22.07.2
v22.07.1
v22.07
v22.06.2
v22.06.1
v22.06
v22.04.2
v22.04.1
v22.04
v22.03.1
v22.03
v22.02.3
v22.02.2
v22.02.1
v22.02
v21.12.5
v21.12.4
v21.12.3
v21.12.2
v21.12.1
v21.12
v21.11.3
v21.11.2
v21.11.1
v21.11
v21.10.3
v21.10.2
v21.10.1
v21.10
v21.08.6
v21.08.5
v21.08.4
v21.08.3
v21.08.2
v21.08.1
v21.08
v21.05.4
v21.05.3
v21.05.2
v21.05.1
v21.05
v21.04.6
v21.04.5
v21.04.4
v21.04.3
v21.04.2
v21.04.1
v21.04
v0.31.8
v0.31.7
v0.31.6
v0.31.5
v0.31.4
v0.31.3
v0.31.2
v0.31.1
v0.31.0
v0.30.7
v0.30.6
v0.30.5
v0.30.4
v0.30.3
v0.30.2
v0.30.1
v0.30.0
v0.29.3
v0.29.2
v0.29.1
v0.29.0
v0.28.3
v0.28.2
v0.28.1
v0.28.0
v0.27.5
v0.27.4
v0.27.3
v0.27.2
v0.27.1
v0.27
v0.26.4
v0.26.3
v0.26.2
v0.26.1
v0.26.0
v0.25.5
v0.25.4
v0.25.3
v0.25.2
v0.25.1
v0.25.0
v0.24.3
v0.24.2
v0.24.1
v0.24.0
v0.23.2
v0.23.1
v0.23.0
v0.22.0
v0.21.0
v0.20.3
v0.20.2
v0.20.1
v0.20.0
v0.19.0
v0.18.5
v0.18.4
v0.18.3
v0.18.2
v0.18.1
v0.18.0
v0.17.4
v0.17.3
v0.17.2
v0.17.1
v0.17.0
v0.16.3
v0.16.2
v0.16.1
v0.16.0
v0.15.3
v0.15.2
v0.15.1
v0.15.0
v0.14.3
v0.14.2
v0.14.1
v0.14.0
v0.13.1
v0.13.0
v0.12.2
v0.12.1
v0.12.0
v0.11.2
v0.11.1
v0.11.0
v0.10.0
v0.9.3
v0.9.2
v0.9.1
v0.9.0
v0.8.2
v0.8.1
v0.8.0
v0.7.6
v0.7.5
v0.7.4
v0.7.3
0.7.2
v.0.7.1
v0.7.0
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.0
Labels
Clear labels
🎨 Design
📖 Docs Update
🐛 Bug
🐛 Bug
:cat2:🐈 Possible duplicate
💿 Database
☕ Open to discussion
💻 Front-End
🐕 Support
🚪 Authentication
🌍 Translations
🔌 API Task
🏭 Back-End
⛲ Upstream
🔨 Feature Request
🛠️ Enhancement
🛠️ Enhancement
🛠️ Enhancement
❤️ Happy feedback
🔒 Security
🔍 Pending Validation
💆 UX
📝 WYSIWYG Editor
🌔 Out of scope
🔩 API Request
:octocat: Admin/Meta
🖌️ View Customization
❓ Question
🚀 Priority
🛡️ Blocked
🚚 Export System
♿ A11y
🔧 Maintenance
> Markdown Editor
pull-request
Mirrored from GitHub Pull Request
No Label
🐛 Bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/BookStack#2494
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @RixzZ on GitHub (Nov 19, 2021).
Describe the Bug
Using a Docker container with Exim for mailing, rejects requests from BookStack as HELO is not correctly announced when opening the SMTP connection.
This was also reported a while ago on issue #2384, but was closed while waiting for more information.
The error the Exim container throws is:
I have this same setup working with other projects which didn't need any special configuration and sent a HELO correctly, allowing them to send email without problems.
Steps to Reproduce
ixdotai/smtp:latestdocker image (the one I use)..envfile to connect to the SMTP container.Expected Behaviour
BookStack should start the SMTP with a good HELO (maybe the public domain name set on APP_URL could work). An IP is not an option either, since is not a valid HELO.
Screenshots or Additional Context
No response
Exact BookStack Version
v21.11
PHP Version
7.4
Hosting Environment
Ubuntu 20.04, but using docker:
I'll explain my setup:
mailingand every container I want to communicate with it is added to this network. The container is calledsmtpwhich later is referenced on the.envfrom BookStack (since is a DNS resolvable name).wikiand his container name ismain.10.0.1.6is the one the BookStack container uses when connected to the 'mailing' network.This SMTP container is local only so it doesn't have any authentication or SSL termination. This is the configuration set on Bookstack
.env:@ssddanbrown commented on GitHub (Nov 19, 2021):
Hi @RixzZ,
This can come down to how the webserver is configured within the system.
Are you using the linuxserver.io docker container? If so then the following would be relevant:
https://github.com/linuxserver/docker-bookstack/issues/68
@RixzZ commented on GitHub (Nov 19, 2021):
Hi @ssddanbrown,
Yeah, I'm using that container, so that might be the reason it's failing in this case, since the HELO that reaches Exim is
_. I guess they need to fix it on the container then. Thank you.Either way, shouldn't be possible to set it from the application when using SMTP? I think the PHP library used for it might have that ability and it could be a new optional setting on the
.env.@ssddanbrown commented on GitHub (Nov 19, 2021):
Yeah, It's likely possible, and we'd probably be able to use the existing
APP_URLvalue, but it'd tie us down to the lower level library implementation while technical be a change in implementation for a much wider set of users, introducing potential risk to wider set of current configurations without spending time understanding or pre-empting how the change may apply. That's upon then maintaining this change for this specific case going forward. For the level of occurance, where this only pops up in container+exim usage, I'm not sure it's worth the change on the app side.For the linuxserver.io container, pretty sure they expose the nginx config through the
/configfile you should have mounted via a volume.@RixzZ commented on GitHub (Nov 19, 2021):
Okay @ssddanbrown, thank you for considering it 👍🏼