Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bugfix: remove Host from auth header #330

Merged
merged 3 commits into from
Jul 11, 2020
Merged

Bugfix: remove Host from auth header #330

merged 3 commits into from
Jul 11, 2020

Conversation

fronzbot
Copy link
Owner

Description:

Removed host from auth header on requests

Related issue (if applicable): fixes #329

Checklist:

  • Local tests with tox run successfully PR cannot be meged unless tests pass
  • Changes tested locally to ensure platform still works as intended
  • Tests added to verify new code works

@codecov
Copy link

codecov bot commented Jul 11, 2020

Codecov Report

Merging #330 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #330   +/-   ##
=======================================
  Coverage   82.34%   82.34%           
=======================================
  Files           8        8           
  Lines         759      759           
=======================================
  Hits          625      625           
  Misses        134      134           
Flag Coverage Δ
#unittests 82.34% <100.00%> (ø)
Impacted Files Coverage Δ
blinkpy/blinkpy.py 81.53% <100.00%> (ø)
blinkpy/helpers/constants.py 100.00% <100.00%> (ø)

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 5e98f59...151d9b7. Read the comment docs.

@fronzbot fronzbot merged commit bcf0bb5 into master Jul 11, 2020
@fronzbot fronzbot deleted the bugfix branch July 11, 2020 14:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant