Skip to content

Use the JSON serialization of Credentials instead of pickle#189

Merged
sqrrrl merged 1 commit intogoogleworkspace:masterfrom
nickmooney:master
Feb 2, 2021
Merged

Use the JSON serialization of Credentials instead of pickle#189
sqrrrl merged 1 commit intogoogleworkspace:masterfrom
nickmooney:master

Conversation

@nickmooney
Copy link
Copy Markdown

In my email to Jon Dormondy:

I am hoping that we can move from pickle to another serialization format such as json in this example. It's a relatively small thing, but pickle can lead to serious vulnerabilities if mishandled, so it would be nice to nudge users in the direction of using secure serialization frameworks.

Creating this pull request per Jon's response to that email -- will also file internally.

@nickmooney
Copy link
Copy Markdown
Author

I've personally tested the change to the Sheets quickstart, and I tried to apply the exact same changes to other files, but some testing should be performed.

@sqrrrl sqrrrl merged commit 43fc5bc into googleworkspace:master Feb 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants