Skip to content

Commit

Permalink
fix CECSConnection::S3GetLifecycle()
Browse files Browse the repository at this point in the history
GFW-965: Lifecycle Rules: Expired Object Delete Marker checkbox always shows CHECKED even if it is not set
  • Loading branch information
Bob Kast committed Jan 5, 2021
1 parent 9e208d6 commit 0805752
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions ECSUtil/ECSConnection.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7064,6 +7064,7 @@ HRESULT XmlS3LifecycleInfoCB(const CStringW& sXmlPath, void *pContext, IXmlReade
if (sXmlPath.CompareNoCase(XML_S3_LIFECYCLE_INFO_RULE) == 0)
{
pInfo->LastRule.Empty();
pInfo->LastRule.bExpiredDeleteMarkers = false;
}
break;
case XmlNodeType_EndElement:
Expand All @@ -7074,6 +7075,7 @@ HRESULT XmlS3LifecycleInfoCB(const CStringW& sXmlPath, void *pContext, IXmlReade
{
pInfo->pLifecycleInfo->LifecycleRules.push_back(pInfo->LastRule);
pInfo->LastRule.Empty();
pInfo->LastRule.bExpiredDeleteMarkers = false;
}
}
break;
Expand All @@ -7098,6 +7100,7 @@ CECSConnection::S3_ERROR CECSConnection::S3GetLifecycle(LPCTSTR pszBucket, S3_LI
if (!RetData.IsEmpty())
{
XML_S3_LIFECYCLE_INFO_CONTEXT Context;
Context.LastRule.bExpiredDeleteMarkers = false;
Context.pLifecycleInfo = &Lifecycle;
HRESULT hr = ScanXml(&RetData, &Context, XmlS3LifecycleInfoCB);
if (FAILED(hr))
Expand Down
4 changes: 2 additions & 2 deletions ECSUtil/Version.h
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,6 @@

#pragma once

#define ECSUTIL_COPYRIGHT _T("Copyright © 2020 Dell Inc. or its subsidiaries.")
#define ECSUTIL_COPYRIGHT _T("Copyright © 2021 Dell Inc. or its subsidiaries.")

#define ECSUTIL_VERSION "v1.0.7.15"
#define ECSUTIL_VERSION "v1.0.7.16"

0 comments on commit 0805752

Please sign in to comment.