Fix Crash When 'Content-Disposition' Header Is Missing

[aroop]

This pull request addresses an issue in the HTTPStorage class, specifically in the download_file method. Previously, the method would fail and crash if the 'content-disposition' header was missing from the HTTP response. This is a common scenario, especially when dealing with servers that don't include this optional header.

• Modified the download_file method in HTTPStorage to use the get method when accessing resp.headers["content-disposition"]. This will return None instead of raising an exception if the header is missing.
• Added a conditional check to ensure that the content disposition header is not None before attempting to parse it with a regular expression.

This fix will prevent the application from crashing when attempting to download files from servers that do not include a 'content-disposition' header. The application's robustness and reliability will be improved, especially in diverse server environments.

[gadicc]

Thanks, @aroop! A very welcome fix and nicely explained too. Many thanks. :pray: :tada:

[gadicc]

:tada: This PR is included in version 1.6.0 :tada:

The release is available on:

• GitHub release
• v1.6.0

Your semantic-release bot :package::rocket: