Addshore has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/364984 )
Change subject: Add more logging in HttpApiLookup::actuallyGetApiUrl
......................................................................
Add more logging in HttpApiLookup::actuallyGetApiUrl
Change-Id: I9cff2cb9f77f6f33d360580ecd8ff07b9fb51651
---
M src/Remote/MediaWiki/HttpApiLookup.php
1 file changed, 8 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/FileImporter
refs/changes/84/364984/1
diff --git a/src/Remote/MediaWiki/HttpApiLookup.php
b/src/Remote/MediaWiki/HttpApiLookup.php
index ef8bf76..2901696 100644
--- a/src/Remote/MediaWiki/HttpApiLookup.php
+++ b/src/Remote/MediaWiki/HttpApiLookup.php
@@ -44,10 +44,11 @@
/**
* @param string $message
+ * @param array $context
* @throws ImportException
*/
- private function logAndException( $message ) {
- $this->logger->error( $message );
+ private function logAndException( $message, $context = [] ) {
+ $this->logger->error( $message, $context );
throw new ImportException( $message );
}
@@ -86,7 +87,11 @@
}
$this->logAndException(
'Failed to discover API location from: "' .
$sourceUrl->getUrl() . '".' .
- 'Got status code ' . $e->getStatusCode() . '.'
+ $e->getMessage() . '. Got status code ' .
$e->getStatusCode() . '.',
+ [
+ 'statuscode' => $e->getStatusCode(),
+ 'previousmessage' => $e->getMessage(),
+ ]
);
return null; // never reached
}
--
To view, visit https://gerrit.wikimedia.org/r/364984
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9cff2cb9f77f6f33d360580ecd8ff07b9fb51651
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/FileImporter
Gerrit-Branch: master
Gerrit-Owner: Addshore <addshorew...@gmail.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
