Fixed SparkleUpdateInfoProvider ignoring appcast_request_headers #277
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
introduced a subtle bug wherein the SparkleUpdateInfoProvider processor
was effectively ignoring
appcast_request_headers
.get_feed_data()
function, the line callingself.fetch_content()
was givingappcast_query_pairs
as the secondargument (which
get_feed_data()
took as itsheaders=
argument).Since
appcast_query_pairs
are url encoded in the code blockimmediately before this call, all that’s missing was to remove the
extraneous argument with a corrected
headers=
one.