RequestsHTTPTransport does not set headers on session like AIOHTTPTransport does
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 84/100
Piste de recherche
Commencez par gql.transport.requests.RequestsHTTPTransport et la reproduction dans test/test_gql.py, en comparant le comportement de sa session avec celui de gql.transport.aiohttp.AIOHTTPTransport. Exécutez le test ciblé pour le transport Requests et confirmez que les en-têtes de la session correspondent aux en-têtes du constructeur, tout en vérifiant que les tests de transport existants continuent de réussir.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Describe the bug
When creating a transport = RequestsHTTPTransport with specific headers, transport.session.headers doesn't match what was given to the constructor (and is in transport.headers`)
This is at odd with the behaviour of AIOHTTPTransport, where we get transport.session.headers == transport.headers
To Reproduce
Steps to reproduce the behavior:
import pytest
from gql.transport.requests import RequestsHTTPTransport
from gql.transport.aiohttp import AIOHTTPTransport
@pytest.mark.asyncio
async def test_aio_http_transport_session():
transport = AIOHTTPTransport("url", headers={"test": "header"})
await transport.connect()
assert transport.headers == transport.session.headers
def test_requests_http_transport_session():
transport = RequestsHTTPTransport("url", headers={"test": "header"})
transport.connect()
assert transport.headers == transport.session.headers
$ uv run pytest -k test_gql -v 130 ↵ ✹ ✭ issue202/rest-client-headers
================================================= test session starts =================================================
platform linux -- Python 3.12.11, pytest-9.1.1, pluggy-1.6.0 -- /home/shtrom/work/simple-github/.venv/bin/python3
cachedir: .pytest_cache
rootdir: /home/shtrom/work/simple-github
configfile: pyproject.toml
plugins: anyio-4.14.2, responses-0.6.0, aioresponses-0.3.0, mock-3.15.1, asyncio-1.4.0
asyncio: mode=Mode.STRICT, debug=False, asyncio_default_fixture_loop_scope=None, asyncio_default_test_loop_scope=function
collected 34 items / 32 deselected / 2 selected
test/test_gql.py::test_aio_http_transport_session PASSED [ 50%]
test/test_gql.py::test_requests_http_transport_session FAILED [100%]
====================================================== FAILURES =======================================================
________________________________________ test_requests_http_transport_session _________________________________________
def test_requests_http_transport_session():
transport = RequestsHTTPTransport("url", headers={"test": "header"})
transport.connect()
> assert transport.headers == transport.session.headers
E AssertionError: assert {'test': 'header'} == {'User-Agent'... 'keep-alive'}
E
E Left contains 1 more item:
E {'test': 'header'}
E Right contains 4 more items:
E {'Accept': '*/*',
E 'Accept-Encoding': 'gzip, deflate, br, zstd',
E 'Connection': 'keep-alive',...
E
E ...Full output truncated (7 lines hidden), use '-vv' to show
test/test_gql.py:17: AssertionError
=============================================== short test summary info ===============================================
FAILED test/test_gql.py::test_requests_http_transport_session - AssertionError: assert {'test': 'header'} == {'User-Agent'... 'keep-alive'}
===================================== 1 failed, 1 passed, 32 deselected in 0.10s ======================================
Expected behavior
transport.session.headers == transport.headers for RequestHTTPTransport, same as AIOHTTPTransport already does.
System info (please complete the following information):
- OS: Linux x86_64, Ubuntu 26.04
- Python version: 3.14.4
- gql version: 4.0
- graphql-core version: 3.2.11
- Langage dominant
- Python
- Étoiles
- 1.7k
- Forks
- 197
- Merge moyen
- 12 min
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de graphql-python/gql
-
type: feature
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
graphql-python/gql#434 ·
-
type: feature
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
graphql-python/gql#373 · 1 réaction ·
-
type: feature
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
graphql-python/gql#355 · 1 commentaire ·
-
type: feature
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
graphql-python/gql#316 · 5 commentaires · 3 réactions ·
Toutes les issues de graphql-python/gql
Issues similaires
-
json_params_matcher fails on falsy top-level JSON primitives (0, False, "")Peut-être pris @mayureshsonawane17 l’a pris aujourd’hui. OuverteWaiting for: Product Owner
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 5 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour
-
第二章思考题 8：Skill 追加到末尾不必每轮重新计算 KVOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
bojieli/ai-agent-book#1169 ·
Les mainteneurs répondent en général sous 1 jour
-
priority:low ready-for-dev
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
OpenHands/extensions#738 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
micronaut-projects/micronaut-core#13677 ·
Les mainteneurs répondent en général sous 1 jour