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

Fix link with parameter not working with jmeter-plugins-manager #374

Merged
merged 1 commit into from
Mar 23, 2020

Conversation

ubikloadpack
Copy link
Contributor

Hello @undera ,
Our url had a parameter in the end "?c=1584900487" , it seems it does not work with plugins manager, the file written is ubik-jmeter-videostreaming-plugin-6.6.0.jar?c=1584900487 instead of ubik-jmeter-videostreaming-plugin-6.6.0.jar

So this PR removes this parameter.
We will try to provide a fix.

Regards

@codecov
Copy link

codecov bot commented Mar 22, 2020

Codecov Report

Merging #374 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #374   +/-   ##
=========================================
  Coverage     69.36%   69.36%           
  Complexity     2571     2571           
=========================================
  Files           230      230           
  Lines         15457    15457           
  Branches       1567     1567           
=========================================
  Hits          10722    10722           
  Misses         3967     3967           
  Partials        768      768           

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 870638f...b4ae8f9. Read the comment docs.

@undera undera merged commit 563abe9 into undera:master Mar 23, 2020
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.

3 participants