node-fetch/node-fetch

abort and correct the error message when `body.formData()` fails

Open

#1.509 geöffnet am 19. Feb. 2022

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (8.563 Stars) (1.061 Forks)batch import
bughelp wanted

Beschreibung

Reproduction

Steps to reproduce the behavior:

  1. res = new Response('foo=bar').formData()

Expected behavior

reject with TypeError: Failed to fetch and then also abort the response/request.

Additional context for fixer in body.js -> formdata() look at the content-type header, if it's null or not a multipart or a searchparam and then abort the request to not leave it hanging and reject with a type error

Contributor Guide

Tech Stack
javascriptnodejs
Domain
backendapi
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Node.js basicsFetch API knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
The issue requires modifying the `formdata()` method in `body.js` to check the `content type` header and abort the response/request if the content type is not multipart/form data or application/x www form urlencoded. The method should reject with a TypeError. See the reproduction steps: `new Response('foo=bar').formData()`. No linked PRs or assignees, so implementation is open.