<#1784 Add Azure-specific headers when uploading t...
# flyte-github
a
#1784 Add Azure-specific headers when uploading to blob storage Pull request opened by devictr TL;DR This PR fixes an issue causing
pyflyte run --remote
to fail when using Azure. Type ☑︎ Bug Fix ☐ Feature ☐ Plugin Are all requirements met? ☑︎ Code completed ☑︎ Smoke tested ☑︎ Unit tests added ☑︎ Code documentation added • [] Any pending items have an associated Issue Complete description The PUT call to Azure Blob Storage returned an HTTP 400 Error:
Copy code
<?xml version="1.0" encoding="utf-8"?>
<Error>
    <Code>MissingRequiredHeader</Code>
    <Message>
        An HTTP header that's mandatory for this request is not specified.
        RequestId:ebd5dfbc-f01e-001d-5074-cb4f26000000
        Time:2023-08-10T10:24:58.6553767Z
    </Message>
    <HeaderName>x-ms-blob-type</HeaderName>
</Error>
This is a starting point that unblocks me, but there's a discussion to be had regarding where you would like to handle such cases. Some open questions: 1. Should this be done on the client? And if yes, is that the right place? 2. Or should the response that contains the signed URL also specify headers based on configuration options? I have tested this locally by tweaking the class and trying to run a workflow Tracking Issue flyteorg/flyte#3942 Follow-up issue NA OR https://github.com/flyteorg/flyte/issues/ flyteorg/flytekit All checks have passed 30/30 successful checks