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

Send response headers #19

Merged
merged 1 commit into from
Jun 18, 2021
Merged

Send response headers #19

merged 1 commit into from
Jun 18, 2021

Conversation

kuniteru
Copy link
Contributor

Added sending of arbitrary response headers.

Some samplers return information in the response headers that is useful for performance analysis.

For example, the BlazeMeter - Video Streaming (HLS Plugin) sampler returns the length (time) of the video segment in the response header X-MEDIA-SEGMENT-DURATION.
By comparing this value with the response time, testers can determine if testers are getting sufficient performance.

I wrote an article about sending the results of Video Streaming Sampler to Application Insights using azure-backend-listener with this change applied, and displaying the results in Application Insights Workbooks.
https://blog.pnop.co.jp/jmeter-azure-media-services_en/

If you find it effective, I would be happy to you adopt it.
The response header values are assigned to a column starting with "aih.". However, you may change it to more suitable prefix.

Copy link
Owner

@adrianmo adrianmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kuniteru sorry for taking this long to review your PR. You did a great job by adding this new feature. I just tested it myself and it worked like a charm.

Thank you!

@adrianmo adrianmo merged commit b1a21ba into adrianmo:master Jun 18, 2021
@kuniteru
Copy link
Contributor Author

Thank you!

@kuniteru kuniteru deleted the send-headers branch June 19, 2021 18:05
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