From 43432cb70d70fc7fc20b1554e12dcc7c8d72440c Mon Sep 17 00:00:00 2001 From: Sipke Schoorstra Date: Thu, 13 Jun 2024 23:10:10 +0200 Subject: [PATCH] Add response stream to context in DownloadHttpFile In DownloadHttpFile.cs, context data now includes response content's file stream. Changes to DefaultDownloadableManager.cs simplify the return statement when no provider is found, directly returning an empty array. --- src/modules/Elsa.Http/Activities/DownloadHttpFile.cs | 1 + src/modules/Elsa.Http/Services/DefaultDownloadableManager.cs | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/modules/Elsa.Http/Activities/DownloadHttpFile.cs b/src/modules/Elsa.Http/Activities/DownloadHttpFile.cs index c4988a99d..8e62f56fd 100644 --- a/src/modules/Elsa.Http/Activities/DownloadHttpFile.cs +++ b/src/modules/Elsa.Http/Activities/DownloadHttpFile.cs @@ -150,6 +150,7 @@ public class DownloadHttpFile : Activity, IActivityPropertyDefaultValu var responseContentHeaders = new HttpHeaders(response.Content.Headers); context.Set(Response, response); + context.Set(ResponseContent, file?.Stream); context.Set(Result, file); context.Set(StatusCode, statusCode); context.Set(ResponseHeaders, responseHeaders); diff --git a/src/modules/Elsa.Http/Services/DefaultDownloadableManager.cs b/src/modules/Elsa.Http/Services/DefaultDownloadableManager.cs index 5e7ad3fe8..dcd954a72 100644 --- a/src/modules/Elsa.Http/Services/DefaultDownloadableManager.cs +++ b/src/modules/Elsa.Http/Services/DefaultDownloadableManager.cs @@ -24,7 +24,7 @@ public class DefaultDownloadableManager : IDownloadableManager var provider = _providers.FirstOrDefault(x => x.GetSupportsContent(content)); if (provider == null) - return Enumerable.Empty>>(); + return []; options ??= new(); var context = new DownloadableContext(this, content, options, cancellationToken);