• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

ipfs / ipfs-cluster / 2939
65%
master: 65%

Build:
Build:
LAST BUILD BRANCH: feat/expvar
DEFAULT BRANCH: master
Ran 03 Oct 2018 11:06PM UTC
Jobs 1
Files 64
Run time 3s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

03 Oct 2018 10:57PM UTC coverage: 65.389% (+2.2%) from 63.228%
2939

push

travis-ci

hsanjuan
Add endpoints output: improvements and compliance

This straigthens some mistakes with the outputs of the /add endpoints.

Currently, we had exactly the same output format which:

* was not exactly the ipfs API output format but was sort of similar
* made some weird concessions to be compatible (like having a string-type "size")
* was not aligned with Cluster API conventions (lowercase keys)

This corrects all this:

* The Cluster API /add output format now uses the right types and lowercase keys.
* `Hash` is now `Cid`, because the field carries a Cid.
* We copy error handling with request trailers from IPFS, and avoid carrying the
  errors in the output objects.
* The proxy now returns exactly the types as ipfs would
* We add the X-Chunked-Output: 1 header, which is custom and redundant, but
otherwise breaks js-ipfs-api integrations with the /add endpoint.

License: MIT
Signed-off-by: Hector Sanjuan <code@hector.link>

34 of 34 new or added lines in 7 files covered. (100.0%)

6331 of 9682 relevant lines covered (65.39%)

84.7 hits per line

Jobs
ID Job ID Ran Files Coverage
1 2939.1 03 Oct 2018 11:06PM UTC 0
65.39
Travis Job 2939.1
Source Files on build 2939
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #2939
  • b6306a6e on github
  • Prev Build on fix/add-output-format (#2937)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc