diff --git a/src/Common/Auth/OpenIDConnect/JWT/JsonWebKeySet.php b/src/Common/Auth/OpenIDConnect/JWT/JsonWebKeySet.php index e5fde31278c..b0d91c2f704 100644 --- a/src/Common/Auth/OpenIDConnect/JWT/JsonWebKeySet.php +++ b/src/Common/Auth/OpenIDConnect/JWT/JsonWebKeySet.php @@ -80,7 +80,7 @@ public function __construct(ClientInterface $httpClient, $jwks_uri = null, $jwks */ public function getJSONWebKey($kid, $alg) { - $this->logger->debug("Attempting to find web key for kid & alg", ['kid' => $kid, 'alg' => $alg]); + $this->logger->debug("JsonWebKeySet::getJSONWebKey() Attempting to find web key for kid & alg", ['kid' => $kid, 'alg' => $alg]); foreach ($this->jwks as $key) { if ($key->kty === 'RSA') { if (!isset($kid) || $key->kid === $kid) { diff --git a/src/Common/Auth/OpenIDConnect/JWT/RsaSha384Signer.php b/src/Common/Auth/OpenIDConnect/JWT/RsaSha384Signer.php index c79c297bf5f..a224e945979 100644 --- a/src/Common/Auth/OpenIDConnect/JWT/RsaSha384Signer.php +++ b/src/Common/Auth/OpenIDConnect/JWT/RsaSha384Signer.php @@ -103,7 +103,7 @@ public function verify($expected, $payload, $key): bool if ($key instanceof JsonWebKeySet) { $kid = $this->headers['kid'] ?? null; - $this->logger->debug("RsaSha384Signer->verify() attempting to retrieve jwk"); + $this->logger->debug("RsaSha384Signer->verify() attempting to retrieve jwk", ['kid' => $kid]); $jwk = $key->getJSONWebKey($kid, $this->algorithmId()); } else { $key = $key instanceof Key ? $key->contents() : $key; diff --git a/src/RestControllers/RestControllerHelper.php b/src/RestControllers/RestControllerHelper.php index 802e27db642..2fdc32471a6 100644 --- a/src/RestControllers/RestControllerHelper.php +++ b/src/RestControllers/RestControllerHelper.php @@ -278,15 +278,17 @@ public function addOperations($resource, $items, FHIRCapabilityStatementResource } if ($operation == '$export') { + // operation definition must use the operation 'name' + // rest.resource.operation.name must come from the OperationDefinition's code attribute which in this case is 'export' + $definitionName = 'export'; + $operationName = 'export'; if ($resource != '$export') { - $operationName = strtolower($resource) . '-export'; - } else { - $operationName = 'export'; + $definitionName = strtolower($resource) . '-export'; } // define export operation $fhirOperation = new FHIRCapabilityStatementOperation(); - $fhirOperation->setName($operation); - $fhirOperation->setDefinition(new FHIRCanonical('http://hl7.org/fhir/uv/bulkdata/OperationDefinition/' . $operationName)); + $fhirOperation->setName($operationName); + $fhirOperation->setDefinition(new FHIRCanonical('http://hl7.org/fhir/uv/bulkdata/OperationDefinition/' . $definitionName)); $capResource->addOperation($fhirOperation); } else if ($operation === '$bulkdata-status') { $fhirOperation = new FHIRCapabilityStatementOperation();