FLUID-3474: Refreshing the page while a package is being built throws an error: using Safari

Metadata

Source
FLUID-3474
Type
Bug
Priority
Critical
Status
Closed
Resolution
Won't Fix
Assignee
N/A
Reporter
Justin Obara
Created
2010-01-08T08:39:55.000-0500
Updated
2014-03-03T11:20:12.406-0500
Versions
  1. 1.4
Fixed Versions
  1. 1.5
Component
  1. Infusion Builder

Description

Refreshing the page while a package is being built throws an error

Steps to reproduce:

1) Open the Infusion Builder
http://forge.fluidproject.org/infusionBuilder/html/InfusionBuilder.html

2) Select a set of modules that haven't been cached.

3) While the package is being built reload the page.

Notice that a white page with text that says "Error" appears.

Here is the information from the response header.

http://forge.fluidproject.org/infusionBuilder/php/builder.php
Response Headers
Connection:close
Content-Encoding:gzip
Content-Length:50
Content-Type:text/html
Date:Fri, 08 Jan 2010 13:25:58 GMT
Server:Apache/2.2.9 (Debian) PHP/5.2.6-1+lenny4 with Suhosin-Patch
Status:400
Vary:Accept-Encoding
X-Powered-By:PHP/5.2.6-1+lenny4

It appears that the refresh is happening on builder.php instead of InfusionBuilder.html

Environments

Safari 4 (Mac OS 10.6)

Comments

  • Laurel Williams commented 2010-01-08T08:49:02.000-0500

    While I'm not sure exactly what is causing this, I do have an idea of how to aviod it. The logic in builder.php is as follows:
    1) if get['var'] is in the $_GET variable then default to the jsonProcessor.php code - show the download page
    2) if post['selection_choice'], etc. is in the $_POST variable then process the build request
    3) if neither of the above then show an error page (likely someone is trying to hack).

    A better way to handle this might be to default to the first alternative unless the post variables are in place. This will prevent hacking but also return a sensible result in all cases.

  • Justin Obara commented 2014-03-03T11:20:12.398-0500

    builder.fluidproject.org has been taken down and there are no immediate plans to replace it. Currently all builds are expected to be generated directly from the grunt build scripts.

    http://fluid.2324889.n4.nabble.com/Infusion-Builder-issues-tp8926.html