Oathkeeper docs claim a max_request_body_size config on the remote authorizer that doesn't exist in the schema or source
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Documentación
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Área
- documentation
Línea de trabajo
Abre la página de documentación de Oathkeeper sobre authz y localiza la sección del autorizador remoto que contiene el claim max_request_body_size. Verifícalo con spec/config.schema.json, pipeline/authz/remote.go y pipeline/authz/utils.go; se considera terminado cuando se elimine el claim inexacto o se marque claramente la funcionalidad como no disponible.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Preflight checklist
- I could not find a solution in the existing issues, docs, nor discussions.
- I agree to follow this project's Code of Conduct.
- I have read and am following this repository's Contribution Guidelines.
- I have joined the Ory Community Slack.
- I am signed up to the Ory Security Patch Newsletter.
Ory Network Project
No response
Describe the bug
The Oathkeeper documentation page for the authz pipeline
(https://www.ory.com/docs/oathkeeper/pipeline/authz) describes a
max_request_body_size setting on the remote authorizer that does
not exist in the config schema or the actual implementation.
The page states:
The maximum inbound request body size that Ory Oathkeeper buffers
so it can replay the body to the upstream service. The default is
"10MiB".Ory Oathkeeper rejects requests that exceed the limit with HTTP 413
before calling the remote authorizer.
This appears to be incorrect / describes a feature that was never
implemented:
spec/config.schema.jsondefinesconfigAuthorizersRemoteand
configAuthorizersRemoteJSONwith onlyremote,headers,
forward_response_headers_to_upstream,retry(andpayloadfor
the JSON variant) — nomax_request_body_sizeproperty on either.pipeline/authz/remote.goandpipeline/authz/utils.go
(pipeRequestBody) pipe the request body via an unbounded
io.TeeReaderinto abytes.Buffer— no size check, no cap, no
HTTP 413 anywhere in either file.- A GitHub commit search for
MaxRequestBodySizeacross the full
repo history returns zero results — this identifier has never
existed in the codebase.
Suggested fix: remove the claim from the docs page, or if this was a
planned-but-unshipped feature, mark it clearly as not yet available.
Reproducing the bug
Not applicable in the docker/curl sense — this is a documentation
inaccuracy, not a runtime bug. To verify:
- Visit https://www.ory.com/docs/oathkeeper/pipeline/authz and read
theremoteauthorizer section — note themax_request_body_size
/ 10MiB / HTTP 413 claim. - Check spec/config.schema.json on the
masterbranch — search for
max_request_body_size: no match under either theremoteor
remote_jsonauthorizer definitions. - Check pipeline/authz/remote.go and pipeline/authz/utils.go on the
masterbranch — no body size limit logic anywhere in either
file; pipeRequestBody() buffers the body unbounded. - Search commit history for
MaxRequestBodySize— zero results.
Relevant log output
N/A — this is a documentation inaccuracy, not a runtime error. No
logs to provide.
Relevant configuration
N/A — no runtime config triggers this. For reference, the actual
(correct) schema for the affected section, from
spec/config.schema.json on the master branch:
authorizers:
remote:
config:
remote: <url>
headers: {}
forward_response_headers_to_upstream: []
retry: {}
No max_request_body_size property exists here or under remote_json.
Version
master
On which operating system are you observing this issue?
None
In which environment are you deploying?
None
Additional Context
No response
- Lenguaje dominante
- TypeScript
- Estrellas
- 161
- Forks
- 1.9k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
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 ory/docs
-
good first issue help wanted upstream
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
good first issue help wanted upstream
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
-
Shared kratos pages can only belong to one sidebar — OEL readers lose navigation contextQuizá libre de nuevo @hperl la tomó hace 76 días y no hay ningún pull request abierto. Abierto
-
good first issue help wanted upstream
Dificultad 1/5 Menos de una hora Aptitud para principiantes 58/100
Issues similares
-
module-request
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
ports get and web print 'Port N already in use, trying next...' for every busy port they skipAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
appandflow/stim#1604 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
lingdojo/kana-dojo#31060 · 1 comentario · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
SSH workspace restore rewrites relative symlinks into the deleted sync-back staging directoryAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
paperclipai/paperclip#14173 ·
Los mantenedores suelen responder en 1 día
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día