HomePhorge

Fix for #4489 -- apply was using the rest terminus
b397b698314dUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Fix for #4489 -- apply was using the rest terminus

Basing pervasive low-level behaviour changes on the application name isn't a
good idea, but if we're going to do it we need to remember to update the test
when we rename the application. Further, if we optimize around such low-level
tests (as we did when implementing file-streaming) we need to carry the tests
over into the new data paths.

This commit addresses both of these isues, restoring 0.25.x stand-alone file
serving by adjusting the process-name test in the indirector to recognize
"apply" as standalone (and thus not use http to fetch the files) and by adding
a branch to the file streaming code to do the same when fetching the content.

I am not certain that this change is globally correct; there may be other
related problems that will not be fixed (and may in fact be exacerbated) by
this patch.

Details

Provenance
Markus Roberts <Markus@reality.com>Authored on
James Turnbull <james@lovedthanlost.net>Committed on Sep 7 2010, 4:49 AM
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPUe91a8cc97521: [#4462] uncaught Timeout::Error
Branches
Unknown
Tags
Unknown

Event Timeline

James Turnbull <james@lovedthanlost.net> committed rPUb397b698314d: Fix for #4489 -- apply was using the rest terminus (authored by Markus Roberts <Markus@reality.com>).Sep 7 2010, 4:49 AM