FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

random generation of keyring / cryptokey names by ryanmats · Pull Request #786 · GoogleCloudPlatform/python-docs-samples · GitHub

random generation of keyring / cryptokey names - #786

Merged
dpebot merged 2 commits into
masterfrom
kms-keyring-randomly-generated
Feb 8, 2017
Merged

random generation of keyring / cryptokey names#786
dpebot merged 2 commits into
masterfrom
kms-keyring-randomly-generated

Conversation

ryanmats commented Feb 7, 2017
edited by dpebot
Loading

Copy link
Copy Markdown
Contributor

No description provided.

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 7, 2017
ryanmats requested a review from theacodes February 7, 2017 23:04
Comment thread kms/api-client/snippets_test.py Outdated

# Your Google Cloud Platform KeyRing name
KEYRING = 'sample-keyring-43'
KEYRING = ''

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Here's a one-liner for you

KEYRING = ''.join(random.choice(string.ascii_lowercase + string.digits) for _ in range(12))

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

done

Copy link
Copy Markdown
Contributor

@dpebot please merge when travis passes

dpebot commented Feb 8, 2017

Copy link
Copy Markdown
Collaborator

Okay! I'll merge when all statuses are green and all reviewers approve.

dpebot added the automerge Merge the pull request once unit tests and other checks pass. label Feb 8, 2017
dpebot self-assigned this Feb 8, 2017
dpebot merged commit 1d76ef0 into master Feb 8, 2017
dpebot deleted the kms-keyring-randomly-generated branch February 8, 2017 22:41
busunkim96 pushed a commit to googleapis/python-kms that referenced this pull request Jun 4, 2020
…atform/python-docs-samples#786)

* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
rsamborski pushed a commit that referenced this pull request Nov 8, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
rsamborski pushed a commit that referenced this pull request Nov 11, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
rsamborski pushed a commit that referenced this pull request Nov 14, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
dandhlee pushed a commit that referenced this pull request Nov 14, 2022
* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Oct 21, 2023
…atform/python-docs-samples#786)

* random generation of keyring / cryptokey names

* Fixed formatting of keyring name and cryptokey name
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL