Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.0.x #2872

Closed
wants to merge 9 commits into from
Closed

3.0.x #2872

wants to merge 9 commits into from

Conversation

pgjones
Copy link
Member

@pgjones pgjones commented Apr 1, 2024

No description provided.

adamtheturtle and others added 9 commits March 3, 2024 15:39
Without this change, `pyright` with `typeCheckingMode` set to `strict` reports:

```
error: Type of "fields" is partially unknown
    Type of "fields" is "MultiDict[Unknown, Unknown]" (reportUnknownVariableType)
```

when I use this method.
This should ensure that the tests work with Pytest 8 onwards. The
issue appears to be that __subclasses__ "returns a list of all those
references still alive." which could include the RequestRedirect. If
it does include RequestRedirect the tests will fail as it requires an
argument to be constructed. Note this test is not meant for
RequestRedirect.
TypeErrors are now handled in the same manner as ValueErrors.

fixes #2842
Previously if a request had repeated slashes it could match a single
slash route and hence return a redirect response even if merge_slashes
was False.

Additionally setting the merge_slashes attribute of the map after
initialisation had no affect, compounding this problem.
This could happen for example with `min=0;max=500` as the `;` is not a
word character everything before it is ignored in the regex during the
finditer call. This then lefts the user confused as the `min=0;` was
silently ignored.
As this solves a publishing error.
@pgjones pgjones closed this Apr 1, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants