tree ac7ecadb55fd81beef111d3a98c6593fcaafaa3c
parent 6f581ec3584cd9b4556a114d30f7a39c63ec64ec
author Jiaqing Zhao <jiaqing.zhao@intel.com> 1649840517 +0800
committer Ed Tanous <ed@tanous.net> 1650035588 +0000

CertificateService: Enhance error handling for ReplaceCertificate

Current implementation of the ReplaceCertificate API always returns
ResourceNotFound if any error occurs when calling DBus, regardless of
whether the certificate to be replaced exists or not. This patch checks
the error code and only return ResourceNotFound when the object path
does not exist, otherwise returns InternalError.

Tested:
* Replace HTTPS certificate at /redfish/v1/Managers/bmc/NetworkProtocol
  /HTTPS/Certificates/0 (Invalid URL) returns ResourceNotFound.
* Replace HTTPS certificate at /redfish/v1/Managers/bmc/NetworkProtocol
  /HTTPS/Certificates/1 with CertificateString not containing private
  key returns InternalError.

Change-Id: I67f6014c3856c192b4141e6a92f173a9a8c8189e
Signed-off-by: Jiaqing Zhao <jiaqing.zhao@intel.com>