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

fix: cat deeply nested file #1920

Merged
merged 2 commits into from
Mar 14, 2019
Merged

fix: cat deeply nested file #1920

merged 2 commits into from
Mar 14, 2019

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Mar 12, 2019

The exporter was exporting the file, but the filter was filtering it out because the exported path includes only file.txt but the filter was expecting path/file.txt.

So when you cat /ipfs/QmHash/file.txt it worked, but when you cat /ipfs/QmHash/path/file.txt it did not because the exported file had path file.txt but the filter was looking for path/file.txt.

Depends on:

The exporter was exporting the file, but the filter was filtering it out because the exported path includes only `file.txt` but the filter was expecting `path/file.txt`.

So when you cat `/ipfs/QmHash/file.txt` it worked, but when you cat `/ipfs/QmHash/path/file.txt` it did not.

License: MIT
Signed-off-by: Alan Shaw <[email protected]>
@ghost ghost assigned alanshaw Mar 12, 2019
@ghost ghost added the status/in-progress In progress label Mar 12, 2019
@alanshaw alanshaw requested a review from achingbrain March 12, 2019 14:00
@alanshaw
Copy link
Member Author

Merging because the CI error is unrelated:

  1) cli
       dht
         query:
     Error: Command failed: /Users/travis/build/ipfs/js-ipfs/src/cli/bin.js dht query QmRhsb2HQUGSrZstoMqiYunqgDzw566igxtgdTwxbgrGuQ
connect ECONNREFUSED 127.0.0.1:52581
      at makeError (node_modules/execa/index.js:174:9)
      at Promise.all.then.arr (node_modules/execa/index.js:278:16)
      at process._tickCallback (internal/process/next_tick.js:68:7)

@alanshaw alanshaw merged commit dcb453a into master Mar 14, 2019
@alanshaw alanshaw deleted the fix/cat-deeply-nested-file branch March 14, 2019 08:51
@ghost ghost removed the status/in-progress In progress label Mar 14, 2019
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.

1 participant