Skip to content
This repository has been archived by the owner on Jul 2, 2020. It is now read-only.

Add cloud_platform scope for Stackdriver Error Reporting #55

Merged
merged 11 commits into from
Aug 25, 2017

Conversation

akm
Copy link
Contributor

@akm akm commented Aug 23, 2017

Add a scope "https://www.googleapis.com/auth/cloud-platform" for Stackdriver Error Eeporting .

This PR depends on #54 , so see 4ae7437...features/add_cloud_platform_scope for actual differences.

I'm going to add a feature to customize the scopes on #53 because some user application might depend on the scopes for VMs.

@@ -70,7 +70,8 @@
"https://www.googleapis.com/auth/devstorage.full_control",
"https://www.googleapis.com/auth/pubsub",
"https://www.googleapis.com/auth/logging.write",
"https://www.googleapis.com/auth/monitoring.write"
"https://www.googleapis.com/auth/monitoring.write",
"https://www.googleapis.com/auth/cloud-platform"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missaligned indent?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@minimum2scp Thanks, I fixed it.

@akm akm force-pushed the features/add_cloud_platform_scope branch from 66fd4a1 to 1a0259a Compare August 23, 2017 09:48
@minimum2scp minimum2scp merged commit 64c803c into master Aug 25, 2017
@minimum2scp minimum2scp deleted the features/add_cloud_platform_scope branch August 25, 2017 01:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants