[Bug 61285] New: SMTP Sampler doesn't configure localhost name

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=61285

            Bug ID: 61285
           Summary: SMTP Sampler doesn't configure localhost name
           Product: JMeter
           Version: 3.2
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: michal.rames@gmail.com
  Target Milestone: ---

If a jmeter is run on a computer not connected to a domain, the underlying
JavaMail implementation resolves the computer name for EHLO command. 

According to the https://tools.ietf.org/html/rfc5321 only fully qualified
domain name may be used. Computer name usually isn't a fully qualified domain
name.

As a result the SMTP server may respond with '501 Invalid EHLO'

This can be easily overriden in
src/protocol/mail/org.apache.jmeter.protocol.smtp.sampler.protocol.SendMailCommand.java
in public void prepareMessage():

props.setProperty("mail.smtp.localhost", localhostDomainName);

And set from GUI via SmtpSampler ... 

If you accept this as a bug fix or enhancement request, I am offering to
provide the required code. Let me know what's the best way (e.g. pull request
in GitHub? )

Thanks / Michal

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 61285] SMTP Sampler doesn't configure localhost name

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=61285

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om
           Hardware|PC                          |All

--- Comment #2 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Hello,
Thanks for proposal, to contribute Github is a better way, see:

- http://jmeter.apache.org/building.html

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 61285] SMTP Sampler doesn't configure localhost name

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=61285

Felix Schumacher <fe...@internetallee.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All

--- Comment #1 from Felix Schumacher <fe...@internetallee.de> ---
Thanks for the report. I would be happy if you would prepare a patch.

I think it would be useful to broaden the scope a bit and let the user specify
any key value pair she likes for the properties in props.

-- 
You are receiving this mail because:
You are the assignee for the bug.