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

vdn-har-convertor-jmeter-tool v3.2 #648

Merged
merged 2 commits into from
Mar 30, 2024
Merged

vdn-har-convertor-jmeter-tool v3.2 #648

merged 2 commits into from
Mar 30, 2024

Conversation

vdaburon
Copy link
Contributor

vdn-har-convertor-jmeter-tool v3.1, use library har-to-jmeter-convertor-2.2 that correct bugs.

  • Remove the header 'Content-length' because the length is computed by JMeter when the request is created.
  • POST or PUT could have query string and body with content so add query string to the path.
  • Set Content Encoding to UFT-8 for POST or PUT method and request Content-Type : application/json.
  • Add body data content in Record.xml for PUT and PATCH methods.

vdn-har-convertor-jmeter-tool v3.1, use library har-to-jmeter-convertor-2.2 that correct bugs.
Copy link

codecov bot commented Mar 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.92%. Comparing base (09d34c0) to head (285a063).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #648      +/-   ##
============================================
+ Coverage     68.89%   68.92%   +0.02%     
- Complexity     2607     2608       +1     
============================================
  Files           230      230              
  Lines         15769    15769              
  Branches       1629     1629              
============================================
+ Hits          10864    10868       +4     
+ Misses         4092     4089       -3     
+ Partials        813      812       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vdaburon
Copy link
Contributor Author

Sorry i find a bug for POST with Query String and body JSON.

Don't push this version

vdn-har-convertor-jmeter-tool version 3.2 (remove v3.1)
@vdaburon
Copy link
Contributor Author

OK vdn-har-convertor-jmeter-tool version 3.2

@undera undera changed the title vdn-har-convertor-jmeter-tool v3.1 vdn-har-convertor-jmeter-tool v3.2 Mar 30, 2024
@undera undera merged commit ef5f339 into undera:master Mar 30, 2024
3 checks passed
@vdaburon
Copy link
Contributor Author

Thanks Andrey

besessener pushed a commit to besessener/jmeter-plugins that referenced this pull request May 15, 2024
* vdn-har-convertor-jmeter-tool v3.1

vdn-har-convertor-jmeter-tool v3.1, use library har-to-jmeter-convertor-2.2 that correct bugs.

* vdn-har-convertor-jmeter-tool version 3.2

vdn-har-convertor-jmeter-tool version 3.2 (remove v3.1)
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