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.

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

Encode the personal part of email addresses in SMTP Sampler #644

Closed

Conversation

FSchumacher
Copy link
Contributor

Description

Often those personal parts contain umlauts. Try to find these
and let MimeUtility encode those as quoted encodings.

Motivation and Context

Bugzilla Id: 65149

How Has This Been Tested?

Used a simple test plan and a dummy smtp server.

Screenshots (if appropriate):

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the code style of this project.
  • I have updated the documentation accordingly.

@FSchumacher FSchumacher force-pushed the bug65149-encode-email-addresses branch from 7e80827 to 3f2d9ec Compare February 21, 2021 10:05
Often those personal parts contain umlauts. Try to find these
and let MimeUtility encode those as quoted encodings.

Bugzilla Id: 65149
@FSchumacher FSchumacher force-pushed the bug65149-encode-email-addresses branch from 3f2d9ec to fd8efc5 Compare February 21, 2021 10:13
@codecov-io
Copy link

codecov-io commented Feb 21, 2021

Codecov Report

Merging #644 (fd8efc5) into master (ed20f5a) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #644      +/-   ##
============================================
- Coverage     55.46%   55.44%   -0.02%     
+ Complexity    10181    10179       -2     
============================================
  Files          1046     1046              
  Lines         64147    64157      +10     
  Branches       7265     7267       +2     
============================================
- Hits          35580    35573       -7     
- Misses        26060    26080      +20     
+ Partials       2507     2504       -3     
Impacted Files Coverage Δ Complexity Δ
...ache/jmeter/protocol/smtp/sampler/SmtpSampler.java 2.89% <0.00%> (-0.18%) 2.00 <0.00> (ø)
...ter/protocol/http/proxy/SamplerCreatorFactory.java 27.90% <0.00%> (-13.96%) 4.00% <0.00%> (-1.00%)
...ter/protocol/http/proxy/DefaultSamplerCreator.java 54.12% <0.00%> (-0.52%) 28.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed20f5a...fd8efc5. Read the comment docs.

@asfgit asfgit closed this in d10b0e5 Feb 27, 2021
asfgit pushed a commit that referenced this pull request Mar 8, 2021
That Exception can't be thrown in that private method. Part of
new code, that was introduced to encode the personal part of
email addresses in SMTP Sampler

Bugzilla Id: 65149
Relates to #644 on github
kkalinin pushed a commit to kkalinin/jmeter that referenced this pull request Mar 11, 2021
Often those personal parts contain umlauts. Try to find these
and let MimeUtility encode those as quoted encodings.

Bugzilla Id: 65149
Closes apache#644 on github
kkalinin pushed a commit to kkalinin/jmeter that referenced this pull request Mar 11, 2021
That Exception can't be thrown in that private method. Part of
new code, that was introduced to encode the personal part of
email addresses in SMTP Sampler

Bugzilla Id: 65149
Relates to apache#644 on github
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants