Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Trying to install an invalid extension file gives "Unknown internal error" message #3174

Closed
gruehle opened this issue Mar 19, 2013 · 7 comments
Assignees
Milestone

Comments

@gruehle
Copy link
Member

gruehle commented Mar 19, 2013

Steps to repro:

  1. Try to install a file that is not a valid extension. For example:
    https://github.com/adobe/brackets/blob/master/src/styles/images/active_back.png

Results
Error dialog that says: "Installation failed. Unknown internal error."

Expected
A more helpful error message :-)

@gruehle
Copy link
Member Author

gruehle commented Mar 19, 2013

This same error occurs if you try to install a valid .zip file that does not contain a valid extension.

@peterflynn
Copy link
Member

FBNC @gruehle -- pull #3178 included a fix for this (without realizing it -- I'd thought this bug was a different case of errors not getting bubbled up to the UI properly)

@gruehle
Copy link
Member Author

gruehle commented Mar 20, 2013

@dangoor @peterflynn - Well, the error is better now, but still not great.

Trying to install "https://dl.dropbox.com/u/17644987/README.md" gives the error: "/var/folders/6z/3_mbdyv124xbql2flrtwj5lh0000gn/T//brackets.download1 is not a valid zip file."

The "is not a valid zip file" part is good, but the filename is pretty scary looking :-)

@ghost ghost assigned dangoor and peterflynn Mar 20, 2013
@gruehle
Copy link
Member Author

gruehle commented Mar 20, 2013

Removing the FBNC label. I don't think this is a sprint-stopper, but I'll keep the Sprint 22 milestone on for now.

@peterflynn
Copy link
Member

Hmm, yeah... showing the exact local filename made more sense back when we were downloading to the parent of the extensions folder. But even then I think it wasn't that useful in the final production code because we delete the file the instant the error message is shown. I'll put up a pull to just not show filenames in cases like this.

@njx
Copy link
Contributor

njx commented Mar 21, 2013

FBNC (again) to @gruehle

@ghost ghost assigned gruehle Mar 21, 2013
@gruehle
Copy link
Member Author

gruehle commented Mar 21, 2013

Much better error messages. Thanks! Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants