Skip to content

[Authlib] Update to 1.7.1#15741

Open
donbarbos wants to merge 1 commit intopython:mainfrom
donbarbos:bump-Authlib-1.7.1
Open

[Authlib] Update to 1.7.1#15741
donbarbos wants to merge 1 commit intopython:mainfrom
donbarbos:bump-Authlib-1.7.1

Conversation

@donbarbos
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 9, 2026

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

Copy link
Copy Markdown
Collaborator

@srittau srittau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, two comments regarding undocumented Anys.

@dataclass
class EndpointRequest:
request: OAuth2Request
client: Any = None
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a comment explaining the Any. Also, we can use Any | None to slight tighten it.

Comment on lines +18 to +20
def create_response(self, validated_request: EndpointRequest) -> tuple[int, Any, list[Incomplete]] | None: ...
def create_endpoint_response(self, request: OAuth2Request) -> tuple[int, Any, list[Incomplete]] | None: ...
def __call__(self, request: OAuth2Request) -> tuple[int, Any, list[Incomplete]] | None: ...
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants