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

Include S3 object content type in attributes #676

Conversation

garyttierney
Copy link
Contributor

The ContentTypeHeaderChecker is invoked after the NameChecker, but the contentType is never set when the attributes are fetched from S3. This patch fixes that.

@jcoyne
Copy link
Contributor

jcoyne commented Aug 14, 2024

@garyttierney Can you rebase this on the latest version of develop? That ought to fix the test failures.

@garyttierney garyttierney force-pushed the bugfix/s3-content-type branch from 9ce4ddc to 155c3e4 Compare August 15, 2024 00:38
@garyttierney
Copy link
Contributor Author

@jcoyne yep, done!

@glenrobson
Copy link
Contributor

Glen to look at suggested issues:

  • Add specific imports back into the code
  • Now this is fixed what happens to the existing behaviour which may work on the file name extension

Glen to look into this issue.

@glenrobson
Copy link
Contributor

Closing in favour of #700

@glenrobson glenrobson closed this Sep 11, 2024
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.

3 participants