Skip to content
Snippets Groups Projects
Verified Commit 172f01a6 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3310510 by igorbarato, Anchal_gupta: Harden error logging of OEmbed...

Issue #3310510 by igorbarato, Anchal_gupta: Harden error logging of OEmbed thumbnail fetching logic (YouTube errors contain special characters)

(cherry picked from commit c36f03ae)
parent 7d191e44
Branches
Tags
24 merge requests!8394[warning] array_flip(): Can only flip STRING and INTEGER values, when saving a non-revisionable custom content entity,!7780issue 3443822: fix for 'No route found for the specified format html. Supported formats: json, xml.',!5013Issue #3071143: Table Render Array Example Is Incorrect,!4848Issue #1566662: Update module should send notifications on Thursdays,!4792Issue #2230689: Remove redundant "Italic" style,!4220Issue #3368223: Link field > Access to internal links is not checked on display.,!3884Issue #3356842,!3870Issue #3087868,!3812Draft: Issue #3339373 by alexpott, andypost, mondrake:...,!3686Issue #3219967 against 9.5.x,!3683Issue #2939397: Clearing AliasManager cache with root path raises warning,!3543Issue #3344259: Allow ajax dialog to have focus configurable,!3356Issue #3209129: Scrolling problems when adding a block via layout builder,!2921Issue #1383696: Allow a custom HTML element to be selected for a grouping field,!2857Issue #3314541: Remove unnecessary fill from SVG icon for the "Media Library" CKEditor 5 button — enabling dark mode support in contrib,!2280Issue #3280415: Metapackage Generator Breaks Under Composer --no-dev,!2205Quote all names in the regions section.,!2050Issue #3272969: Remove UnqiueField constraint.,!1956Issue #3268872: hook_views_invalidate_cache not called when a view is deleted,!1893Issue #3217260: Add a way to make media captions not editable in CKEditor,!1459Issue #3087632: menu_name max length is too long,!878Issue #3221534: throw an exception when IDs passed to loadMultiple() are badly formed,!866Issue #2845319: The highlighting of the 'Home' menu-link does not respect query strings and fragment identifiers,!204Issue #3040556: It is not possible to react to an entity being duplicated
...@@ -447,7 +447,9 @@ protected function getLocalThumbnailUri(Resource $resource) { ...@@ -447,7 +447,9 @@ protected function getLocalThumbnailUri(Resource $resource) {
} }
} }
catch (TransferException $e) { catch (TransferException $e) {
$this->logger->warning($e->getMessage()); $this->logger->warning('Failed to download remote thumbnail file due to "%error".', [
'%error' => $e->getMessage(),
]);
} }
catch (FileException $e) { catch (FileException $e) {
$this->logger->warning('Could not download remote thumbnail from {url}.', [ $this->logger->warning('Could not download remote thumbnail from {url}.', [
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment