app.add_url_rule() should look for OPTIONS methods in a case-insensitive manner

[reteptilian]

add_url_rule() has logic that says that if the user has not included OPTIONS in their methods, flask will automatically provide an implementation.

Unfortunately, this logic is only triggered if the method string is OPTIONS in all upper case. So if you've included lower case "options" in your methods you will still end up with the flask-provided options functionality.

I think it should be a case-insensitive comparison since all the rest of the routing is all case-insensitive.

[mhall1]

Not exactly the most efficient solution, but I don't think the list will be long enough for it to matter. I thought it better to avoid touching anything that could be used downstream.

[untitaker]

@mhall1 It's safe to uppercase everything in methods. Werkzeug does this anyway in Rule.__init__.

[mhall1]

@untitaker okay, fixed. Thanks for your feedback.