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

Remove useless getObjectRetention call … #156

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

votdev
Copy link
Contributor

@votdev votdev commented Mar 24, 2023

… because the information is delivered by the headObject call. The problem here was that the Traefic ingress is incorrect configured in regards to the Access-Control-Expose-Headers header. Because of that the headers X-Amz-Object-Lock-Mode and X-Amz-Object-Lock-Retain-Until-Date where not included in the response.

Related to: https://github.com/aquarist-labs/s3gw/issues/410 and s3gw-tech/s3gw-charts#89

Describe your changes

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • CHANGELOG.md has been updated should there be relevant changes in this PR

@votdev votdev added kind/quality Quality improvements, Refactoring, Automation via CI, E2E, Integration, CLI or REST API area/ui User Interface labels Mar 24, 2023
@votdev votdev added this to the v0.14.0 milestone Mar 24, 2023
@votdev votdev requested a review from jhmarina March 24, 2023 13:31
@votdev votdev self-assigned this Mar 24, 2023
@votdev votdev marked this pull request as ready for review March 27, 2023 08:56
…ivered by the headObject call. The problem here was that the Traefic ingress is incorrect configured in regards to the `Access-Control-Expose-Headers` header. Because of that the headers `X-Amz-Object-Lock-Mode` and `X-Amz-Object-Lock-Retain-Until-Date` where not included in the response.

Signed-off-by: Volker Theile <vtheile@suse.com>
@votdev votdev force-pushed the improve_headobject branch from 3d0ecf2 to 63c6ccb Compare March 27, 2023 11:30
@votdev votdev merged commit da8bb73 into s3gw-tech:main Mar 27, 2023
@votdev votdev deleted the improve_headobject branch March 27, 2023 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui User Interface kind/quality Quality improvements, Refactoring, Automation via CI, E2E, Integration, CLI or REST API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant