You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jmeter.apache.org by lucamaro <gi...@git.apache.org> on 2016/04/18 10:55:04 UTC

[GitHub] jmeter pull request: Bug 59345 - SMTPSampler connection leak

GitHub user lucamaro opened a pull request:

    https://github.com/apache/jmeter/pull/198

    Bug 59345 - SMTPSampler connection leak

    Move trasport closing command into finally block in order to avoid SMTP connection leak when sending email fails. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/lucamaro/jmeter trunk

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/jmeter/pull/198.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #198
    
----
commit 679462fd88fea0c1a27b924858bf236650dab654
Author: Luca Maragnani <lu...@fastweb.it>
Date:   2016-04-13T12:01:56Z

    Close SMTP connection in case of exception

commit cfcccdc44885ac403d6b0f6f2215a2b89e5beaa8
Author: Luca Maragnani <lu...@fastweb.it>
Date:   2016-04-13T12:01:56Z

    Close SMTP connection in case of exception

commit 37946b1659cb4c99a568e440da61aafa7b3b2b5b
Author: Luca Maragnani <lu...@fastweb.it>
Date:   2016-04-18T08:44:30Z

    Merge branch 'trunk' of https://github.com/lucamaro/jmeter into trunk

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] jmeter pull request: Bug 59345 - SMTPSampler connection leak

Posted by vlsi <gi...@git.apache.org>.
Github user vlsi commented on the pull request:

    https://github.com/apache/jmeter/pull/198#issuecomment-211285422
  
    Thanks for the change. It looks good.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] jmeter pull request: Bug 59345 - SMTPSampler connection leak

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/jmeter/pull/198


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---