Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strip unnecessary payload in AD root cred rotation example #8160

Merged
merged 3 commits into from
Jan 18, 2020

Conversation

tyrannosaurus-becks
Copy link
Contributor

@tyrannosaurus-becks tyrannosaurus-becks commented Jan 15, 2020

Will merge after #8154 is in and I've updated the change to match.

@jasonodonnell jasonodonnell self-requested a review January 15, 2020 21:04
jasonodonnell
jasonodonnell previously approved these changes Jan 15, 2020
Copy link
Contributor

@jasonodonnell jasonodonnell left a comment

Choose a reason for hiding this comment

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

LGTM!

kalafut
kalafut previously approved these changes Jan 15, 2020
@tyrannosaurus-becks
Copy link
Contributor Author

Thanks! When it's time to merge it, I'll also strip the payload from here, Jason pointed it out.

@tyrannosaurus-becks tyrannosaurus-becks merged commit 96b2b39 into master Jan 18, 2020
@tyrannosaurus-becks tyrannosaurus-becks deleted the strip-payload-line branch January 18, 2020 00:49
catsby added a commit that referenced this pull request Jan 21, 2020
* master: (34 commits)
  Use Shamir as KeK when migrating from auto-seal to shamir (#8172)
  changelog++
  ssh backend: support at character in role name (#8038)
  Fix typo in YAML markup (#8194)
  Fix typo (#8192)
  Fix k8s injector examples (#8179)
  update dependencies, patch nextjs config (#8184)
  Strip unnecessary payload in AD root cred rotation example (#8160)
  New Website! (#8154)
  Update CHANGELOG.md
  changelog++
  Fix panic when listener fails to startup (#8174)
  Create network layer abstraction to allow in-memory cluster traffic (#8173)
  Update test var name and tidy
  Factor out mysqlhelper so we can create mysql docker containers in other tests. (#8167)
  changelog++
  changelog++
  Pull wrapping creation to a var (#8137)
  ldap, okta: fix renewal when login policies are empty (#8072)
  Update CHANGELOG.md
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants