pep8speaks icon indicating copy to clipboard operation
pep8speaks copied to clipboard

fix: query request helpers headers

Open Mr-Sunglasses opened this issue 9 months ago • 0 comments

fix: #227 fix: #216

Before behaviour

Code:

def query_request(query=None, method="GET", **kwargs):
    """
    Queries like /repos/:id needs to be appended to the base URL,
    Queries like https://raw.githubusercontent.com need not.

    full list of kwargs see http://docs.python-requests.org/en/master/api/#requests.request
    """

    if query[0] == "/":
        query = BASE_URL + query

    request_kwargs = {
        "headers": {"Authorization": f"Bearer {GITHUB_TOKEN}"}
    }

    
    request_kwargs.update(**kwargs)
    # return requests.request(method, query, **request_kwargs)
    return request_kwargs


def follow_user(user):
    """Follow the user of the service"""
    headers = {"Content-Length": "0"}
    query = f"/user/following/{user}"
    return query_request(query=query, method='PUT', headers=headers)

Output:

{'headers': {'Content-Length': '0'}}

After Changes

{'headers': {'Authorization': 'Bearer TOKEN', 'Content-Length': '0'}}

Mr-Sunglasses avatar Apr 29 '24 23:04 Mr-Sunglasses