search

cbeer / solr_wrapper

Wrap your tests with Solr 5+
MIT License
23 stars 20 forks source link

Fixes cheksum failure when :md5sum option is a URL. #114

Closed dkinzer closed 6 years ago

dkinzer commented 6 years ago

According to the current docs md5sum is either a path or URL to a file containing an md5sum.

When the md5sum is a URL either it is treated as an md5sum string failing checksum test (because it's a URL) or it is not used and default md5sum url for the default mirror is used instead. In either case there is a check sum failure.

This commit fixes that issue by overriing the default md5url when md5sum is a url.

dkinzer commented 6 years ago

I think that maybe there is a missing md5sum method in settings.rb that could possibly fix this also.. not sure.

dkinzer commented 6 years ago

ok, i added the missing method in order to keep backwards compatibility