Azure / Azure/azure-functions-python-worker

[Bug] Python Azure Function worker incorrectly URL encodes cookie value

Abierto
#1,794 1 comentario 2 reacciones 0 asignados Ver en GitHub
area:python-functions bug python python-sdk
Lenguaje dominante
Python
Estrellas
357
Forks
116
Merge medio
32 min
PR fusionados (30 d)
1

Descripción

### Expected Behavior

Azure Function worker using Python FastAPI should not URL encode cookie values

Expected behavior on HTTP Response Headers:
`set-cookie: test="hello//world"; Path=/; SameSite=lax`

e.g. when using Python FastAPI without Azure Functions, it correctly does not URL encode.
````
import uvicorn
from fastapi import FastAPI
from fastapi.responses import PlainTextResponse

app = FastAPI()

@app.get("/")
def http_trigger():
response = PlainTextResponse("", status_code=200)
response.set_cookie(key='test', value='hello//world')

return response

if __name__ == "__main__":
uvicorn.run(app, host="127.0.0.1", port=7071)

````

Run `curl -v http://localhost:7071/` and response is correct as `set-cookie: test="hello//world"; Path=/; SameSite=lax`

### Actual Behavior

Actual incorrect behavior on HTTP Response Headers, cookie value is incorrect and unexpectedly URL encoded

`Set-Cookie: test=hello%2F%2Fworld; domain=; path=/; samesite=lax`

Tested bug with
* Python 3.12 with FastAPI
* Azure Functions Core tools 4.3.0 locally
* Deployed to production Azure Function

### Steps to Reproduce

1. Run `func start` or deploy to Azure using sample code provided below
2. Send a test HTTP request e.g. `curl -v http://localhost:7071/`
3. Azure Functions worker incorrectly URL encodes the response cookie value in the `Set-Cookie` header

### Relevant code being tried

```shell
import azure.functions as func
from fastapi import FastAPI
from fastapi.responses import PlainTextResponse

app = FastAPI()
func_app = func.AsgiFunctionApp(app=app, http_auth_level=func.AuthLevel.ANONYMOUS)

@app.get("/")
def http_trigger():

response = PlainTextResponse("", status_code=200)
response.set_cookie(key='test', value='hello//world')

return response
```

### Relevant log output

```shell
N/A
```

### requirements.txt file

```shell
azure-functions
fastapi
```

### Where are you facing this problem?

Production Environment (explain below)

### Function app name

Private

### Additional Information

_No response_

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza con la ruta del adaptador ASGI creada por func.AsgiFunctionApp y compara su gestión de las cabeceras de respuesta con la salida de PlainTextResponse.set_cookie de FastAPI. Reprodúcelo con func start y curl -v usando el valor hello//world; se considera terminado cuando la cabecera Set-Cookie conserva el valor de cookie esperado y una comprobación de regresión lo cubre.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
azure, fastapi, python
Área
api, backend
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.