@@ -293,9 +293,9 @@ apr_status_t porter_handle_upload(porter_upload_request_t *ur, apreq_param_t *p) if (size == 0) { // There was no file, or at least it had no name, so let's - // treat it like a text area - PORTER_LOG("Appears there was no file, passing control to porter_handle_parameter"); - return porter_handle_parameter(ur, p); + // just skip it. + PORTER_LOG("Appears there was no file, skipping the parameter"); + return APR_SUCCESS; } PORTER_LOG("Appears there was a file, continuing"); mod_porter.c 3a77a621bf32db8c8e4f6cff86bf61ec1e7bafe6 Michael Koziarski michael@koziarski.com http://github.com/actionrails/modporter/commit/b5c949503ef20d29fc14412c93cb8e4ed6e1e993 b5c949503ef20d29fc14412c93cb8e4ed6e1e993 2009-08-02T21:38:52-07:00 2009-08-02T21:38:52-07:00 Don't pass empty strings for empty file upload inputs. The values are useless but their presence trips up some plugins. 028027959d0359993564885c6725c044973fba32 Michael Koziarski michael@koziarski.com