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.

Sign up for GitHub

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

Jump to bottom

Initial greeting on client connect contains package name #163

Open
lord-alfred opened this issue Aug 4, 2019 · 2 comments · May be fixed by #224 or #220
Open

Initial greeting on client connect contains package name #163

lord-alfred opened this issue Aug 4, 2019 · 2 comments · May be fixed by #224 or #220

Comments

@lord-alfred
Copy link
Contributor 

lalf@ubuntu:~$ telnet localhost 25
Trying ::1...
Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
220 testsrv.com SMTP Guerrilla(1.5.1-90-gf9c4965) #1 (1) 2019-08-04T13:40:52+03:00

Why explicitly show the client that this package is used? Why not pretend Postfix?
Just sooner or later, the smartest system of sending letters will begin to filter domains with such an answer without even transferring any data further.

Maybe it makes sense to change this behavior? For example, make the setting in the configuration file.
@flashmob
Copy link
Owner

flashmob commented Aug 5, 2019

Sounds good. PR to make this configurable would be welcome.

@vladtreny
Copy link

Sounds good. PR to make this configurable would be welcome.

Hello,
Thank you for your work.

Is it possible (as for now) to hide or configure this line SMTP Guerrilla?

@lord-alfred lord-alfred mentioned this issue Dec 12, 2020
Open
batterystaples added a commit to batterystaples/go-guerrilla that referenced this issue Feb 13, 2021
@batterystaples
Added greeting_type configuration option (flashmob#163)
31c3895
@batterystaples batterystaples linked a pull request Feb 13, 2021 that will close this issue
Open
@cgroschupp cgroschupp linked a pull request Apr 27, 2021 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@flashmob @lord-alfred @vladtreny