Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GCS: improve mime-type detection #179

Merged
merged 1 commit into from
Sep 28, 2020
Merged

GCS: improve mime-type detection #179

merged 1 commit into from
Sep 28, 2020

Conversation

thomas-advantitge
Copy link
Contributor

As suggested by @drakkan in #178, better mime type detection than used in the google-cloud-go underlying package.

Example which previously gave application/zip as mime type:
image

@codecov
Copy link

codecov bot commented Sep 28, 2020

Codecov Report

Merging #179 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #179   +/-   ##
=======================================
  Coverage   99.33%   99.33%           
=======================================
  Files          32       32           
  Lines        5379     5379           
=======================================
  Hits         5343     5343           
  Misses         19       19           
  Partials       17       17           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 55515fe...8d34b38. Read the comment docs.

@drakkan drakkan changed the title GCS mime-type detection GCS: improve mime-type detection Sep 28, 2020
@drakkan drakkan merged commit cd56039 into drakkan:master Sep 28, 2020
@drakkan
Copy link
Owner

drakkan commented Sep 28, 2020

Thank you

@thomas-advantitge thomas-advantitge deleted the mime-types branch September 29, 2020 07:59
@drakkan
Copy link
Owner

drakkan commented Apr 18, 2024

Hello @thomas-advantitge

thank you again for this contribution. We have recently updated our contribution policy and we now require signing a CLA. Agreeing to the CLA clarifies your rights as a contributor and at the same time protects SFTPGo and its users.

More details here

You can use the following URL to sign the CLA

https://cla-assistant.io/drakkan/sftpgo?pullRequest=179

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants