684134 - file module can't read from resource:// URIs

Status: RESOLVED WONTFIX

(Add-on SDK Graveyard :: General, enhancement, P3)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

While trying to work on bug 682631, I wrote a sample extension to test out the new async readAll() function. I put a text file in the extension's data/ directory, and then tried to read it using file.readAll(data.url("textfile.txt"), callback);

This failed and threw an error:
[Exception... "Component returned failure code: 0x80520001 (NS_ERROR_FILE_UNRECOGNIZED_PATH) [nsILocalFile.initWithPath]"  nsresult: "0x80520001 (NS_ERROR_FILE_UNRECOGNIZED_PATH)"  location: "JS frame :: resource://jid1-osjbqfful5kyag-at-jetpack-api-utils-lib/securable-module.js -> resource://jid1-osjbqfful5kyag-at-jetpack-api-utils-lib/file.js :: MozFile :: line 64"  data: no]


I then tried it using the original sync read() function, and it threw the same error.

Comment 1

[Erik Vold [:erikvold] (please needinfo? me)]

The sdk/url module has a toFilename method which could be used here I think.

Comment 2

[Jordan Santell [:jsantell] [@jsantell]]

I think we're moving towards deprecating io/file -- is this still necessary?

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #2)
> I think we're moving towards deprecating io/file -- is this still necessary?

I'll pass that question over to Irakli. :)

Comment 4

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Yes we'll be moving to io/fs but it will have a same issue. data.url("textfile.txt") is not a file path, as a matter of fact it may even not have path as it will be under compressed .xpi file.

Now data.load("textfile.txt") can already return content of that file, unfortunately performing sync IO, but
if you want to do it async there's also:
https://github.com/mozilla/addon-sdk/blob/master/lib/sdk/net/url.js#L38