Project

General

Profile

Bug #4653

oembed logger warnings under udrulning på version 5.0.1

Added by Christian Vandel 2 months ago. Updated 9 days ago.

Status:
Reviewed
Priority:
High
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Driftsvedligehold - Teknisk tilgængelighed

Description

Udrulning af sites på version 5.0.1 udløser på nogle sites følgende fejl:

preg_match(): Compilation failed: regular expression is too large at offset 55002 oembed.module:182 [warning]

Linien gentages et antal gange og optræder i forbindelse med ding_page_update_7004 og ding_news_update_7007. Samtidig optræder der dublerede loglinier for afviklingen af  ding_page_update_7004 og ding_news_update_7007.

Umiddelbart kunne det godt se ud som om det er dette issue, som er på spil: https://www.drupal.org/project/oembed/issues/2739023

---

[aegir@localhost] out: Performed update: ding_page_update_7004                                     [ok]
[aegir@localhost] out: Performed update: ding_page_update_7004                                     [ok]
[aegir@localhost] out: Performed update: ding_page_update_7004                                     [ok]
[aegir@localhost] out: Performed update: ding_page_update_7004                                     [ok]
[aegir@localhost] out: Performed update: ding_page_update_7004                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: preg_match(): Compilation failed: regular expression is too large at   [warning]
[aegir@localhost] out: offset 55002 oembed.module:182
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
[aegir@localhost] out: Performed update: ding_news_update_7007                                     [ok]
 

 

History

#1 Updated by Rolf Madsen 2 months ago

  • Status changed from New to Ready for development
  • Assignee set to Christel Krabbenhøft
  • Target version set to Release 31-2 - Bug fixes (7.x-5.?.?)

@Christel skal vi ikke få ITK til at tage et kig på den?

Måske det har noget  at gøre med Paragraphs implementeringen.

#2 Updated by Christel Krabbenhøft about 2 months ago

  • Assignee changed from Christel Krabbenhøft to Lene Lykke Rafn

#3 Updated by Jesper Kristensen about 2 months ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Lene Lykke Rafn to Gitte Barlach

Hej

Der er lidt udforinger her. Jeg tror den patch vil fix problemet, men vi kan ikke bruge den i den udgave der ligger på drupal.org, da vi bruger sidste release af modulet (rc1) fra 2013. Men koden er blevet ændret med en række ting siden da (sidste i maj 2019) og den patch der linkes til er til denne kode.

Så skal vi port patch tilbaget til rc1 eller skal vi køre dev udgaven af modulet og patch? Jeg er ikke 110% sikker på hvad oembed bruges til i DDB CMS (er det kun video)?

#4 Updated by Simon Holt about 1 month ago

Jeg er ikke 110% sikker på hvad oembed bruges til i DDB CMS (er det kun video)?

Så vidt jeg kan se bruges modulet ifb. med oembed feltet på ting_objects. Vi har vores eget custom virtuelle felttype, men anvender funktioner fra oembed modulet til bl.a. at bestemme provideren ud fra den online URL der leveres på posten fra brønden. Se modulet ting_oembed.

Det er vist nok mest for at embedde indhold fra ekurser. Ved ikke om andre Online URL fra brønden understøtter oembed.

#5 Updated by Christian HerningBib about 1 month ago

Så vidt jeg ved implementerede Reload oEmbed-modulet i forbindelse med et projekt, der går ud på at der ved visning af en post fra ekurser hentes html, som bl.a. inkluderer youtube-iframes, fra ekurser.nu's oembed-tjeneste, så ekurset vises i selve posten. Projektet er stadig aktivt, selvom vi jvf. issue #4123 har haft problemer med at få resultatet vist i postvisningen. Jeg mener at Reload i sin tid nævnte at modulet også kunne bruges i andre sammenhænge, men tror ikke at det er sket.

 

#6 Updated by Jesper Kristensen about 1 month ago

Grunden til at jeg spørge er jeg mangle lidt at kunne test om jeg "bare" skal gå med `dev` versionen eller vi skal have lavet patch bagud.

 

@gitte har du en ide om vi har sådan en post ved aakb jeg kan bruge som test?

#8 Updated by Jesper Kristensen about 1 month ago

  • Status changed from Need more info to Needs code review

#9 Updated by Gitte Barlach 10 days ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#10 Updated by Jørgen Nielsen 9 days ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

reviewet og godkendt

Also available in: Atom PDF