ghmo / google-refine

Automatically exported from code.google.com/p/google-refine
0 stars 0 forks source link

Blank browser shown when non-GZIP format is detected during import #155

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Try to import corrupted GZIP project file or non-GZIP project file.
2. Browser shows blank screen and goes no further.

What is the expected output? What do you see instead?
A warning page or redirect back to home page with warning in same area as "no 
file chosen", to account for possible corrupted GZIP files or not in proper 
GZIP format.

However, Not in GZIP format exception is caught and displayed in command window 
history.

19:29:18.034 [   import-project_command] Importing existing project using new ID
 1610312782436 (0ms)
java.io.IOException: Not in GZIP format
        at java.util.zip.GZIPInputStream.readHeader(GZIPInputStream.java:137)
        at java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:58)
        at java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:68)
        at com.google.refine.io.FileProjectManager.importProject(FileProjectMana
ger.java:97)
        at com.google.refine.commands.project.ImportProjectCommand.internalImpor
t(ImportProjectCommand.java:91)
        at com.google.refine.commands.project.ImportProjectCommand.doPost(Import
ProjectCommand.java:41)
        at com.google.refine.RefineServlet.service(RefineServlet.java:130)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:820)
        at org.mortbay.jetty.servlet.ServletHolder.handle(ServletHolder.java:511
)
        at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(Servlet
Handler.java:1166)
        at org.mortbay.servlet.UserAgentFilter.doFilter(UserAgentFilter.java:81)

        at org.mortbay.servlet.GzipFilter.doFilter(GzipFilter.java:132)
        at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(Servlet
Handler.java:1157)

Original issue reported on code.google.com by thadguidry on 13 Oct 2010 at 12:46

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r1469.

Original comment by dfhu...@gmail.com on 13 Oct 2010 at 4:51

GoogleCodeExporter commented 8 years ago
Fixed by r1469.

Original comment by dfhu...@google.com on 13 Oct 2010 at 4:51

GoogleCodeExporter commented 8 years ago

Original comment by tfmorris on 18 Sep 2012 at 2:58