Skip to content

Commit ff9ee76

Browse files
committed
Need not check that response is a SAML_Response
It will throw an exception if it's not. And psr-4 transition breaks expected class name
1 parent 26d0c27 commit ff9ee76

1 file changed

Lines changed: 1 addition & 9 deletions

File tree

www/attributequery.php

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -78,15 +78,7 @@
7878
try {
7979
$response = sendQuery($dataId, $data['url'], $nameId, $attributes_to_send, $attributeNameFormat, $src, $dst);
8080
} catch (Exception $e) {
81-
throw new SimpleSAML_Error_Exception('[attributeaggregator] Error in sending query. ' .$e);
82-
}
83-
84-
85-
/* Getting the response */
86-
SimpleSAML_Logger::debug('[attributeaggregator] attributequery - getting response');
87-
88-
if (!($response instanceof SAML2_Response)) {
89-
throw new SimpleSAML_Error_Exception('Unexpected message received in response to the attribute query.');
81+
throw new SimpleSAML_Error_Exception('[attributeaggregator] Got an exception while performing attribute query. Exception: '.get_class($e).', message: '.$e->getMessage());
9082
}
9183

9284
$idpEntityId = $response->getIssuer();

0 commit comments

Comments
 (0)