astro / astro/node-oembed

Crash

Offen
#2 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
38
Forks
15
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

I had a crash when I tried to fetch the following URL: http://www.tatw.co.uk/podcast.xml
I also use embed.ly.

Stack trace
/Users/pronebird/dev/homejs/node_modules/oembed/lib/oembed.js:115
attrs.rel === 'alternate' &&
^
TypeError: Cannot read property 'rel' of undefined
at Object.DiscoveryHandler.writeTag (/Users/pronebird/dev/homejs/node_modules/oembed/lib/oembed.js:115:11)
at Parser$writeHandler [as writeHandler](/Users/pronebird/dev/homejs/node_modules/oembed/node_modules/htmlparser/lib/htmlparser.js:444:20)
at Parser$parseTags [as parseTags](/Users/pronebird/dev/homejs/node_modules/oembed/node_modules/htmlparser/lib/htmlparser.js:384:8)
at Parser$parseChunk [as parseChunk](/Users/pronebird/dev/homejs/node_modules/oembed/node_modules/htmlparser/lib/htmlparser.js:95:8)
at Discovery.write (/Users/pronebird/dev/homejs/node_modules/oembed/lib/oembed.js:78:17)
at IncomingMessage.exports.pump (util.js:496:21)
at IncomingMessage.EventEmitter.emit (events.js:126:20)
at IncomingMessage._emitData (http.js:359:10)
at HTTPParser.parserOnBody [as onBody](http.js:123:21)
at Socket.socketOnData [as ondata](http.js:1367:20)
16 Nov 23:22:59 - [nodemon] app crashed - waiting for file changes before starting...

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Reproduziere den Absturz mit http://www.tatw.co.uk/podcast.xml und untersuche lib/oembed.js:115, indem du dem DiscoveryHandler.writeTag-Aufruf im Stacktrace folgst. Bestätige, dass das Abrufen dieser URL die Anwendung nicht mehr zum Absturz bringt und dass das oEmbed-Discovery-Verhalten weiterhin intakt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, node.js
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.