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

Use HTTPS for NSW Rural Fire feed #7

Merged
merged 1 commit into from Jul 3, 2019

Conversation

Projects
None yet
3 participants
@davidjb
Copy link
Contributor

commented Jul 3, 2019

This small fix uses HTTPS to secure load the feed, over insecure http as it was before. The same feed is delivered, just over this secure protocol for improvements in security, data integrity and privacy.

@coveralls

This comment has been minimized.

Copy link

commented Jul 3, 2019

Pull Request Test Coverage Report for Build 36

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.602%

Totals Coverage Status
Change from base Build 34: 0.0%
Covered Lines: 398
Relevant Lines: 412

💛 - Coveralls
@exxamalte

This comment has been minimized.

Copy link
Owner

commented Jul 3, 2019

Thank you for this contribution ⭐️

@exxamalte exxamalte merged commit b9fcb2e into exxamalte:master Jul 3, 2019

3 checks passed

Hound No violations found. Woof!
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 96.602%
Details
@exxamalte

This comment has been minimized.

Copy link
Owner

commented Jul 3, 2019

I'll release a new version of the library later today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.