Default app-wide automatic_options to False, as it fits Sanic's minim…

…al feel better.

Fixed a failing test caused by app-wide automatic_options implementation.
Bumped version.

sanic_cors/core.py

@@ -50,7 +50,7 @@
                        supports_credentials=False,
                        max_age=None,
                        send_wildcard=False,
-                       automatic_options=True,
+                       automatic_options=False,
                        vary_header=True,
                        resources=r'/*',
                        intercept_exceptions=True,

sanic_cors/decorator.py

@@ -105,6 +105,7 @@ def cross_origin(app, *args, **kwargs):
     _options = kwargs
 
     def decorator(f):
+        nonlocal _options
         LOG.debug("Enabling %s for cross_origin using options:%s", f, _options)
 
         # Sanic does not have the same automatic OPTIONS handling that Flask does,
@@ -124,6 +125,7 @@ def decorator(f):
         #     f.provide_automatic_options = False
 
         async def wrapped_function(req, *args, **kwargs):
+            nonlocal _options
             # Handle setting of Sanic-Cors parameters
             options = get_cors_options(app, _options)
 

sanic_cors/version.py

@@ -1 +1 @@
-__version__ = '0.4.1.3'
+__version__ = '0.4.1.4'

tests/extension/test_app_extension.py

@@ -223,6 +223,7 @@ def setUp(self):
         CORS(self.app, resources=r'/api/*',
              headers='Content-Type',
              expose_headers='X-Total-Count',
+             automatic_options=False,
              origins='http://bar.com')
 
         @self.app.route('/api/v1/foo', methods=['GET', 'HEAD', 'OPTIONS'])
@@ -234,7 +235,7 @@ def exposed2(request):
             return json({"success": True})
 
         @self.app.route('/api/v1/special', methods=['GET', 'HEAD', 'OPTIONS'])
-        @cross_origin(self.app, origins='http://foo.com')
+        @cross_origin(self.app, origins='http://foo.com', automatic_options=True)
         def overridden(request):
             return json({"special": True})