Add app engine credentials #46

theacodes · 2016-10-21T23:35:13Z

Closes #36

google/auth/app_engine.py

+from google.auth import credentials
+from google.auth import exceptions
+
+


google/auth/app_engine.py

+                not available.
+        """
+        # pylint: disable=unused-argument
+        if not app_identity:


google/auth/app_engine.py

+        Returns:
+            bool: True if there are no scopes set otherwise False.
+        """
+        return True if not self._scopes else False


google/auth/app_engine.py

+        Raises:
+            EnvironmentError: If the App Engine APIs are unavailable.
+        """
+        if not app_identity:


google/auth/app_engine.py

+                  credentials.Credentials):
+    """App Engine standard environment credentials.
+
+    These credentials use the App Engine App Idenity API to obtain access


google/auth/app_engine.py

+try:
+    from google.appengine.api import app_identity
+except ImportError:
+    app_identity = None


google/auth/app_engine.py

+            service_account_id (str): The service account ID passed into
+                :func:`google.appengine.api.app_identity.get_access_token`.
+                This is not required as the default application service account
+                ID will be used.


google/auth/app_engine.py

+            google.auth.exceptions.RefreshError: If the App Engine APIs are
+                not available.
+        """
+        # pylint: disable=unused-argument


google/auth/app_engine.py

+
+    @property
+    def requires_scopes(self):
+        """Checks if the credentials requires scopes.


tests/test_app_engine.py

+@pytest.fixture
+def app_engine(app_identity_mock):
+    from google.auth import app_engine
+    reload_module(app_engine)


tests/test_app_engine.py

+def app_engine(app_identity_mock):
+    from google.auth import app_engine
+    reload_module(app_engine)
+    yield app_engine


tests/test_app_engine.py

+@pytest.fixture
+def app_engine_no_apis():
+    from google.auth import app_engine
+    reload_module(app_engine)


tests/test_app_engine.py

+
+
+class TestCredentials(object):
+    def test_default_state(self, app_engine):


tests/test_app_engine.py

+    def test_refresh(self, now_mock, app_engine, app_identity_mock):
+        token = 'token'
+        ttl = 100
+        app_identity_mock.get_access_token.return_value = (token, ttl)


google/auth/app_engine.py

+        self._service_account_id = service_account_id
+
+    def refresh(self, request):
+        """Refresh the access token and scopes.


google/auth/app_engine.py

+        """Refresh the access token and scopes.
+
+        Args:
+            request (google.auth.transport.Request): Unused.


google/auth/app_engine.py

+            scopes=scopes, service_account_id=self._service_account_id)
+
+    def sign_bytes(self, message):
+        """Signs the given message.


theacodes · 2016-10-24T16:52:26Z

@dhermes @nathanielmanistaatgoogle PTAL, should be good to go.

dhermes · 2016-10-24T16:53:53Z

LGTM

Add app engine credentials

a48769a

theacodes added this to the 1.0.0 milestone Oct 21, 2016

theacodes assigned dhermes Oct 21, 2016

nathanielmanistaatgoogle reviewed Oct 22, 2016

View reviewed changes

dhermes reviewed Oct 22, 2016

View reviewed changes

Address review comments

71c7418

nathanielmanistaatgoogle reviewed Oct 24, 2016

View reviewed changes

Address review comments

ef9d9db

theacodes merged commit 0471475 into master Oct 24, 2016

theacodes deleted the gae-credentials branch October 24, 2016 17:01

theacodes unassigned dhermes Oct 11, 2018

		from google.auth import credentials
		from google.auth import exceptions



		class TestCredentials(object):
		def test_default_state(self, app_engine):

Add app engine credentials #46

Add app engine credentials #46

Conversation

theacodes commented Oct 21, 2016

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

theacodes commented Oct 24, 2016

dhermes commented Oct 24, 2016