jakarta-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 50046] SMTPSampler partially ignores the plain body checkbox
Date Thu, 07 Oct 2010 10:49:58 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=50046

Sebb <sebb@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
         Resolution|                            |WONTFIX

--- Comment #5 from Sebb <sebb@apache.org> 2010-10-07 06:49:54 EDT ---
(In reply to comment #4)
> Thank you very much for the quick reply.
> In regard to comment 3, I also saw that same error in my own tests. I believe
> your solution message.setText(""); is the correct one.
> 
> From what I see in the code, the need for the "plain body" checkbox is
> doubtful. Am I correct that we can achieve the exact same behavior of the check
> box simply by not adding any text to the message area and not add any
> attachment?

No.

The check-box affects the format of the message.

> By selecting the checkbox, I expected JMeter to create a plain body and not use
> the text in the message area. If we use the text in there, then it is not plain
> body anymore.

Yes, it is still plain body - as opposed to multipart/mixed.

> So I believe the correct behaviour should rather be:
> 
> •  If checkbox Send plain body is checked and Attach files is empty 
>     ==> SMTP sampler sends a message with content message.setText("")

It does that if you leave the message field empty.

> •  If checkbox Send plain body is checked and Attach files is not empty
>    ==> SMTP sampler send a message with a "send-as-is" for attach files
> (mailBody is ignored)

It does that, so long as the message field is empty.

> Summarizing, I see no reason to add message text if we select plain body. This
> is what I attempted to do with my patch.

Your patch would mean one could only use multipart/mixed, even if there were no
files.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@jakarta.apache.org
For additional commands, e-mail: notifications-help@jakarta.apache.org


Mime
View raw message