fetchResource loads actual items from the API due to wrong queryParameter
@soyuka ya está trabajando en esto.
Desde el 8/11/2024.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
API Platform version(s) affected: 2.6.8
Description
The API doc parser uses fetch() via the fetchResource() method to load API metadata, and fails to apply the query parameter { itemsperpage: 0 } correctly to limit the request to only metadata and no API items.
fetchResource() tries to set the query parameter via a secondary options parameter, however fetch() doesn't support that - params have to be part of the url instead.
// Current code:
fetchJsonLd(
resourceUrl,
Object.assign({ itemsPerPage: 0 }, options)
)
// Fixed code:
fetchJsonLd(
resourceUrl + "?itemsPerPage=0",
options
)
How to reproduce
Possible Solution
Replace the Object.assign() call with an URL including the query parameter, possibly by building an URL:
const url = new URL(resource.url);
const params = new URLSearchParams(["itemsPerPage", 0]);
url.search = params.toString();
fetchJsonLd(url, options)
Additional Context
Please excuse that my code examples are in plain JS, not TS, since I'm not familiar enough with typescript.
- Lenguaje dominante
- TypeScript
- Estrellas
- 113
- Forks
- 80
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de api-platform/api-doc-parser
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
api-platform/api-doc-parser#158 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
api-platform/api-doc-parser#130 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
api-platform/api-doc-parser#129 · 1 comentario ·
-
api-platform/api-doc-parser#125 · 1 comentario · 1 asignado ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 42/100
api-platform/api-doc-parser#113 ·
Todos los issues de api-platform/api-doc-parser
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
bug v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
modelcontextprotocol/inspector#2458 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
railmapgen/rmp-gallery#4068 ·
-
Mend: dependency security vulnerability status: needs triage 🕵️♀️
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
carbon-design-system/ibm-products#9907 ·