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

Enhances MailReaderSampler forget top headers #2127

Closed
asfimport opened this issue Jun 20, 2008 · 2 comments
Closed

Enhances MailReaderSampler forget top headers #2127

asfimport opened this issue Jun 20, 2008 · 2 comments

Comments

@asfimport
Copy link
Collaborator

Michael Bolliger (Bug 45244):
We test a system with the mail reader, which doesn't cache the headers and sign the message 2 times. First time the header which is taken with top and the second time when the MailReader retieves the whole message.This is a problem because, the boundary of the header is different from the message body. In this case it ends in a Boundary-Exception.

this propleme can resolved with the mail properties:

mail.pop3.disabletop=true
mail.pop3.forgettopheaders=true

I have patched the MailReader with adding a checkbox called "Enable header caching". The source based on the MailReader from #2111.
The default value is false, which means the properties on top will be set for starting the session and the pop session forget the header which is taken with top.

With this patch it's not possible to enable and disable the header caching for all samplers with one option.

This could be resolved with java properties for example:

starting jmeter with java properties like :mailReader.mail.pop3.disabletop=true
and the MailReader could check if any java property is set for him. When the MailReader finds a property it can now do something with it. In this case set the properties for the pop session to forget the top headers.

OS: Linux

Depends on:

@asfimport
Copy link
Collaborator Author

Michael Bolliger (migrated from Bugzilla):
Created attachment MailReaderSampler.tar: MailReaderSampler as .tar

@asfimport
Copy link
Collaborator Author

Sebb (migrated from Bugzilla):
It took me a while to realise that this enhancement relates to code that is not part of any JMeter versions.

If you want to add to existing enhancement requests, please either update the original request, or make it very clear at the start that the enhancement relates to proposed code.

In this case, the easiest would be to provide a new patch for #2111.

Closing accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant