Rebased against new 0.23.0rc2 (apache#127)

piptools_requirements3.txt

@@ -6,17 +6,13 @@
 #
 appdirs==1.4.3
 asn1crypto==0.23.0        # via cryptography
-certifi==2017.7.27.1
 cffi==1.9.1               # via cryptography
 cryptography==1.9
 enum34==1.1.6             # via cryptography
-idna==2.5
+idna==2.5                 # via cryptography
 ipaddress==1.0.18         # via cryptography
-ndg-httpsclient==0.4.2
 packaging==16.8
-pyasn1==0.2.2
 pycparser==2.17           # via cffi
-pyopenssl==16.2.0
 pyparsing==2.1.10         # via packaging
 six==1.10.0
 

requirements.in

@@ -130,7 +130,3 @@ s3werkzeugcache==0.1.1
 
 # oauth
 flask-oauthlib
-
-# pydruid
-pydruid==0.4.0
-

requirements.txt

@@ -18,25 +18,22 @@ boto3==1.4.4              # via s3werkzeugcache
 boto==2.47.0
 botocore==1.5.95          # via boto3, s3transfer
 celery[sqs]==3.1.25
-certifi==2017.7.27.1
+certifi==2017.7.27.1      # via requests
 cffi==1.9.1
 chardet==3.0.4            # via requests
 click==6.7                # via flask
-configparser==3.5.0       # via flake8
+configparser==3.5.0
 coverage==4.4.1           # via pytest-cov
 cryptography==1.9
 docutils==0.14            # via botocore
 enum34==1.1.6
-enum==0.4.6               # via pydruid
 flake8-tidy-imports==1.0.6
 flake8==3.3.0
 flask-cache==0.13.1       # via s3werkzeugcache
 flask-oauthlib==0.9.4
 flask-script==2.0.5
 flask==0.12.2             # via flask-cache, flask-oauthlib, flask-script
-funcsigs==1.0.2           # via mock
 future==0.16.0            # via pyhive
-futures==3.1.1            # via s3transfer
 gevent==1.2.1
 greenlet==0.4.12
 gunicorn==19.7.1
@@ -54,25 +51,19 @@ lyft-stdlib==113.0.0
 lyftsettings==0.1.0       # via lyft-requests
 markupsafe==1.0           # via jinja2
 mccabe==0.6.1             # via flake8
-mock==2.0.0               # via pytest-mock
 mysqlclient==1.3.12
-ndg-httpsclient==0.4.2
 oauthlib==2.0.6           # via flask-oauthlib, requests-oauthlib
 packaging==16.8
 pathtools==0.1.2          # via watchdog
-pbr==3.1.1                # via mock
 protobuf==3.2.0           # via lyft-requests
 psutil==5.4.0             # via rainbow-saddle
 psycopg2==2.7.3.2
 py==1.4.34                # via pytest
-pyasn1==0.2.2
 pycodestyle==2.3.1        # via flake8
 pycparser==2.17
 pycurl==7.43.0
-pydruid==0.4.0
 pyflakes==1.5.0           # via flake8
 pyhive==0.5.0
-pyopenssl==16.2.0
 pyparsing==2.1.10
 pytest-cov==2.5.1
 pytest-mock==1.1
@@ -83,7 +74,7 @@ pyyaml==3.12
 rainbow-saddle==0.4.0
 redis==2.10.6
 requests-oauthlib==0.8.0  # via flask-oauthlib
-requests==2.17.3          # via lyft-requests, pydruid, requests-oauthlib
+requests==2.17.3          # via lyft-requests, requests-oauthlib
 s3transfer==0.1.11        # via boto3
 s3werkzeugcache==0.1.1
 six==1.10.0

requirements3.txt

@@ -18,7 +18,7 @@ boto3==1.4.4              # via s3werkzeugcache
 boto==2.47.0
 botocore==1.5.95          # via boto3, s3transfer
 celery[sqs]==3.1.25
-certifi==2017.7.27.1
+certifi==2017.7.27.1      # via requests
 cffi==1.9.1
 chardet==3.0.4            # via requests
 click==6.7                # via flask
@@ -52,21 +52,18 @@ lyftsettings==0.1.0       # via lyft-requests
 markupsafe==1.0           # via jinja2
 mccabe==0.6.1             # via flake8
 mysqlclient==1.3.12
-ndg-httpsclient==0.4.2
 oauthlib==2.0.6           # via flask-oauthlib, requests-oauthlib
 packaging==16.8
 pathtools==0.1.2          # via watchdog
 protobuf==3.2.0           # via lyft-requests
 psutil==5.4.0             # via rainbow-saddle
 psycopg2==2.7.3.2
 py==1.4.34                # via pytest
-pyasn1==0.2.2
 pycodestyle==2.3.1        # via flake8
 pycparser==2.17
 pycurl==7.43.0
 pyflakes==1.5.0           # via flake8
 pyhive==0.5.0
-pyopenssl==16.2.0
 pyparsing==2.1.10
 pytest-cov==2.5.1
 pytest-mock==1.1

superset_config.py

@@ -97,12 +97,13 @@ def has_access(self, permission_name, view_name):
         if tom_request_key:
             if tom_request_key == SECRET_KEY:
                 logging.info('TOM request is authorized')
-                return True
+                return tom_request_key == SECRET_KEY
             logging.warning('TOM request is unauthorized')
 
         return super(LyftSecurityManager, self).has_access(permission_name,
                                                            view_name)
 
+
 CUSTOM_SECURITY_MANAGER = LyftSecurityManager
 
 SUPERSET_WEBSERVER_TIMEOUT = 120