Project

General

Profile

Bug #4653

oembed logger warnings under udrulning på version 5.0.1

Added by Christian Vandel 10 months ago. Updated 22 days ago.

Status:
Reviewed
Priority:
High
Assignee:
Target version:
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 10 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 9 months ago

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

#3 Updated by Jesper Kristensen 9 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 9 months 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 9 months 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 9 months 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 9 months ago

  • Status changed from Need more info to Needs code review

#9 Updated by Gitte Barlach 8 months ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#10 Updated by Jørgen Nielsen 7 months ago

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

reviewet og godkendt

#11 Updated by Kasper Garnæs 5 months ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

#12 Updated by Kasper Garnæs 5 months ago

Merge af dette PR har fået alle vores builds til at fejle efterfølgende. Så vidt jeg kan se applier patchen ikke længere. Derfor har jeg reverted ændringen.

Fejlen blev sandsynligvis ikke fundet fordi CircleCI af uransagelige årsager ikke har kørt på PR'et.

 

#13 Updated by Gitte Barlach 4 months ago

Hej Gitte

Jeg har ikke set dem i forbindelse med 6.0.1, men det er nok snarere en konsekvens af at der ikke er blevet afviklet updates som kan trigge problemet. Eftersom fejlen fik samme update til at køre flere gange er jeg bestemt ikke tryg ved den. Jeg ville nok overveje om det var muligt at opdatere modulet, hvis den version der benyttes er fra 2013!

 

Venlig hilsen
Christian

 

#14 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Jesper Kristensen to Jørgen Nielsen

#15 Updated by Christel Krabbenhøft 4 months ago

  • Assignee changed from Jørgen Nielsen to Kasper Garnæs
  • Target version changed from Release 31-2 - Bug fixes (7.x-5.?.?) to Release 32 - Bugfixes

#16 Updated by Kasper Garnæs 4 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Kasper Garnæs to Jesper Kristensen

Opdateret PR der genintroducerer patchen: https://github.com/ding2/ding2/pull/1641

 

Jeg ville nok overveje om det var muligt at opdatere modulet, hvis den version der benyttes er fra 2013!

Jeg har bevidst ikke opdateret modulet eftersom der endnu ikke er tagget en release af det siden.

 

Jeg har ikke set dem i forbindelse med 6.0.1, men det er nok snarere en konsekvens af at der ikke er blevet afviklet updates som kan trigge problemet.

Enig. Problemet viser sig i forbindelse med opdateringer til indhold, der er implementeret i forbindelse med migreringen til Layout uden grænser.

 

Eftersom fejlen fik samme update til at køre flere gange er jeg bestemt ikke tryg ved den.

Funktionen ding_news_update_7007, der står for migreringen af nyheder til Layout uden grænser, er implementeret således at den kører et antal gange indtil alle nyheder på et side er migreret.

Uden yderligere information mener jeg ikke der er grundlag for at sige at fejlen fik ikke samme update til at køre flere gange.

#17 Updated by Gitte Barlach about 1 month ago

  • Assignee changed from Jesper Kristensen to Jørgen Nielsen

#18 Updated by Jørgen Nielsen 22 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