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

set default value for ExpiredObjectDeleteMarker #1681

Closed
wants to merge 1 commit into from

Conversation

prembhaskal
Copy link

@prembhaskal prembhaskal commented Apr 23, 2022

When ExpiredObjectDeleteMarker is empty, the parsing fails.
Few S3 storage eg. ODF/Noobaa are returning empty tag "<ExpiredObjectDeleteMarker></ExpiredObjectDeleteMarker>" causing parsing to fail.

When ExpiredObjectDeleteMarker is empty, the parsing fails.
Few S3 storage eg. ODF/Noobaa are returning empty tag "<ExpiredObjectDeleteMarker></ExpiredObjectDeleteMarker>" causing parsing to fail.
Copy link
Contributor

@jasdel jasdel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for reaching out with this issue. We cannot take this change as it is because it modifies the SDK's generated code. In addition, could you provide details about the error that you're seeing parsing the ExpiredObjectDeleteMarker member? It probably will be better to close this PR and create a bug issue instead to investigate the issue you're seeing.

@jasdel jasdel added the closing-soon This issue will automatically close in 4 days unless further comments are made. label Apr 28, 2022
@prembhaskal
Copy link
Author

Oh k. I didn't realise this is generated code. Yes i will create a issue with the results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closing-soon This issue will automatically close in 4 days unless further comments are made.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants