Fix HttpAuthenticationType to add callable

[MtkN1]

Add a callable object to the HttpAuthenticationType type definition.

A type checking error occurs when a callable object is given as the auth argument.

Reproduction code

import niquests


def pizza_auth(request: niquests.PreparedRequest) -> niquests.PreparedRequest:
    if request.headers:
        request.headers["X-Pizza"] = "Token"
    return request


def test_callable_auth():
    r = niquests.get("https://httpbin.org/get", auth=pizza_auth)
    print(r.json()["headers"])


if __name__ == "__main__":
    test_callable_auth()

This is allowed at runtime.

niquests/src/niquests/models.py

Lines 615 to 647 in d83ab6b

	def prepare_auth(self, auth: HttpAuthenticationType | None, url: str = "") -> None:
	"""Prepares the given HTTP auth data."""
	assert (
	self.url is not None
	), "Cannot invoke prepare_auth method with incomplete PreparedRequest"
	# If no Auth is explicitly provided, extract it from the URL first.
	if auth is None:
	url_auth = get_auth_from_url(self.url)
	auth = url_auth if any(url_auth) else None
	if auth:
	if isinstance(auth, tuple) and len(auth) == 2:
	# special-case basic HTTP auth
	auth = HTTPBasicAuth(*auth)
	elif isinstance(auth, str):
	auth = BearerTokenAuth(auth)
	if not callable(auth):
	raise ValueError(
	"Unexpected non-callable authentication. Did you pass unsupported tuple to auth argument?"
	)
	if not asyncio.iscoroutinefunction(auth.__call__):
	# Allow auth to make its changes.
	r = auth(self)
	# Update self to reflect the auth changes.
	self.__dict__.update(r.__dict__)
	# Recompute Content-Length
	self.prepare_content_length(self.body)

[Ousret]

The PR itself bring a valid point. I had to make a change to avoid making a whole TypeAlias being a string. Some IDE / type checkers may not like it. It is most importantly a personal design choice.

[MtkN1]

Thanks for the review and fix. I did not know that TypeAlias were allowed to be partial strings. I think it is better that way.