Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

automatic options as separate route#5918

Open
davidism wants to merge 1 commit intomainfrom
automatic-options
Open

automatic options as separate route#5918
davidism wants to merge 1 commit intomainfrom
automatic-options

Conversation

@davidism
Copy link
Member

@davidismdavidism commentedFeb 12, 2026
edited
Loading

Implementprovide_automatic_options as a regular part of the routing and view dispatch, rather than as a special check on every request.

Can be updated to doexcept DuplicateRuleError instead ofException once Werkzeug 3.2 is out. Relies on the duplicate check to not add the internal route multiple times, for example if@app.get and@app.post register different view functions for the same URL. But nothing breaks if the rule is added multiple times, it's just messy.

Adjustsflask routes to no show the_automatic_options endpoint, as that would be really noisy.

Also moves the static view out instead of using a lambda and weakref, to match how this new view is handled.

expands on#5917,fixes#5916

@davidismdavidism added this to the3.2.0 milestoneFeb 12, 2026
@davidismdavidism reopened thisFeb 13, 2026
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.2.0

Development

Successfully merging this pull request may close these issues.

provide_automatic_options is weird

1 participant

@davidism

Comments


[8]ページ先頭

©2009-2026 Movatter.jp