Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Fix return value from HTTP 400s. #929

Merged
merged 1 commit into from
Aug 30, 2018
Merged

Conversation

pattivacek
Copy link
Collaborator

No description provided.

@pattivacek pattivacek requested review from koshelev, OYTIS and lbonn August 30, 2018 08:40
@codecov-io
Copy link

codecov-io commented Aug 30, 2018

Codecov Report

Merging #929 into master will decrease coverage by 0.41%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #929      +/-   ##
==========================================
- Coverage   81.74%   81.33%   -0.42%     
==========================================
  Files         184      185       +1     
  Lines       11755    12443     +688     
==========================================
+ Hits         9609    10120     +511     
- Misses       2146     2323     +177
Impacted Files Coverage Δ
src/sota_tools/ostree_http_repo.cc 1.58% <0%> (ø) ⬆️
src/libaktualizr/primary/sotauptaneclient.cc 72.86% <0%> (-27.14%) ⬇️
src/libaktualizr/http/httpclient.cc 77.12% <0%> (-7.85%) ⬇️
src/sota_tools/ostree_dir_repo.cc 73.52% <0%> (-5.89%) ⬇️
src/libaktualizr/crypto/keymanager.cc 89.65% <0%> (-2.07%) ⬇️
src/libaktualizr/uptane/directorrepository.h 100% <0%> (ø) ⬆️
src/libaktualizr/uptane/uptanerepository.h 100% <0%> (ø) ⬆️
src/libaktualizr/primary/sotauptaneclient.h 100% <0%> (ø) ⬆️
src/libaktualizr/uptane/imagesrepository.h 100% <0%> (ø) ⬆️
src/libaktualizr/bootloader/bootloader.h 100% <0%> (ø)
... and 5 more

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 b116275...cc76b15. Read the comment docs.

@pattivacek pattivacek merged commit fad299b into master Aug 30, 2018
@pattivacek pattivacek deleted the bugfix/fix-http-return-code branch August 30, 2018 10:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants