Add google.auth.credentials - the public interfaces for credentials #8

theacodes · 2016-10-04T21:45:50Z

@nathanielmanistaatgoogle this is the section you indicated you wanted to give input on. PTAL.

theacodes · 2016-10-04T21:53:09Z

I am unsure if I should leave the scopes property as is (completely private) or make it a public read-only property.

dhermes · 2016-10-04T22:08:34Z

I vote for public read-only. I'm holding off a review here until you tell me to look @jonparrott

theacodes · 2016-10-04T22:09:50Z

Sgtm, gonna let Nathaniel chime in first.

On Tue, Oct 4, 2016, 3:08 PM Danny Hermes notifications@github.com wrote:

I vote for public read-only. I'm holding off a review here until you tell
me to look @jonparrott https://github.com/jonparrott

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#8 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc-0omIhmwrF0HlyVl8t7Mu8-MKApks5qws5igaJpZM4KOMdL
.

theacodes · 2016-10-10T20:06:58Z

@nathanielmanistaatgoogle ping, this is now blocking.

dhermes · 2016-10-10T20:09:08Z

@jonparrott rebase?

theacodes · 2016-10-10T20:09:16Z

@dhermes doing it now.

theacodes · 2016-10-10T20:14:26Z

Rebased.

google/auth/credentials.py

+        with :attr:`expiry` set to None is considered to never expire.
+        """
+        now = _helpers.utcnow()
+        if self.expiry is None or self.expiry > now:


google/auth/credentials.py

+            token (Optional[str]): If specified, overrides the current access
+                token.
+        """
+        headers[b'authorization'] = 'Bearer {}'.format(


google/auth/credentials.py

+        This is True if the credentials have a :attr:`token` and the token
+        is not :attr:`expired`.
+        """
+        return self.token is not None and not self.expired


google/auth/credentials.py

+    def has_scopes(self, scopes):
+        """Checks if the credentials have the given scopes.
+
+        .. warning: This method is not guarenteed to be accurate if the


google/auth/credentials.py

+            message (bytes): The message to sign.
+
+        Returns:
+            bytes: The messages cryptographic signature.


google/auth/credentials.py

+        """Create a copy of these credentials with the specified scopes.
+
+        Args:
+            scopes (Union[str, Sequence]): The scope or list of scopes to


google/auth/credentials.py

+
+
+@six.add_metaclass(abc.ABCMeta)
+class ScopedCredentials(object):


google/auth/credentials.py

+        """Refreshes the access token.
+
+        Args:
+            request (google.auth.transport.Request): A callable used to make


google/auth/credentials.py

+    """
+    def __init__(self):
+        self.token = None
+        """str: The bearer token that can be used in HTTP headers to make


theacodes · 2016-10-14T21:07:57Z

@nathanielmanistaatgoogle after discussing with @dhermes offline, I'm gonna go ahead and merge this so we can unblock reviewing some concrete classes.

We can still continue to discuss here and I'll send follow-up PRs to make changes. This is all fluid until 1.0. Unless there is a very significant change I don't anticipate the impact on concrete classes to be large.

theacodes added this to the 1.0.0 milestone Oct 4, 2016

theacodes assigned bjwatson and dhermes Oct 4, 2016

theacodes force-pushed the credentials-abc branch from b00bfcc to 84a8f6b Compare October 10, 2016 20:12

theacodes mentioned this pull request Oct 10, 2016

[Discussion] post 4.0.0 refactor googleapis/oauth2client#597

Closed

nathanielmanistaatgoogle reviewed Oct 10, 2016

View reviewed changes

Jon Wayne Parrott and others added 7 commits October 14, 2016 13:41

Add google.auth.credentials - the public interfaces for credentials

902f761

Fix inline hyperlink

db9aa4c

Fix utcnow usage

78c615c

Address review comments

cd526b9

Make scopes public read-only, restrict type to sequence

2aa80a6

Rename mixin classes

2b83967

Remove unneeded use of byte strings

602be67

theacodes force-pushed the credentials-abc branch from e27eca9 to 602be67 Compare October 14, 2016 20:41

theacodes mentioned this pull request Oct 14, 2016

Add JWT credentials #21

Merged

dhermes approved these changes Oct 14, 2016

View reviewed changes

Address offline review comments

f522fc4

theacodes merged commit 71ce2a0 into master Oct 14, 2016

theacodes deleted the credentials-abc branch October 14, 2016 21:08

theacodes unassigned bjwatson and dhermes Oct 11, 2018



		@six.add_metaclass(abc.ABCMeta)
		class ScopedCredentials(object):

Add google.auth.credentials - the public interfaces for credentials #8

Add google.auth.credentials - the public interfaces for credentials #8

Conversation

theacodes commented Oct 4, 2016

theacodes commented Oct 4, 2016

dhermes commented Oct 4, 2016

theacodes commented Oct 4, 2016

theacodes commented Oct 10, 2016

dhermes commented Oct 10, 2016

theacodes commented Oct 10, 2016

theacodes commented Oct 10, 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.

theacodes commented Oct 14, 2016