SAP / SAP/python-pyodata

ODataHttpRequest : headers inherited from the connection arg are not handled

Open Beginner friendly
#108 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Python
Stars
268
Forks
100
PR merge metrics
No merged PRs in 30d

Description

When executing a request with pyodata's Client, one may need to use some headers, for example for authentication purpose. ODataHttpRequest takes both a Connection argument and a headers keyword-argument, whereas Connection object might already have a headers attribute.
Hence, ODataHttpRequest's init() should merge headers from Connection and from args, overriding the first with the second if necessary.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at ODataHttpRequest.init and inspect how headers from the Connection argument and the headers keyword argument are handled. Confirm the intended precedence described in the issue, then verify that requests preserve inherited headers while allowing explicitly supplied headers to override them.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
62/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.