-
Notifications
You must be signed in to change notification settings - Fork 683
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
Encrypt session cache #8666
Encrypt session cache #8666
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @hawkeyetwolf this looks great!
Could you please apply my suggestions now that we merged your work into our official Github Actions?
Other than that, this LGTM :)
Co-authored-by: Kevin Porras <kporras07@gmail.com>
Co-authored-by: Kevin Porras <kporras07@gmail.com>
@kporras07 Hawkeye passed away in October so I accepted the proposed changes. Thanks again @hawkeyetwolf for all of your contributions to the community. |
@kporras07 once you re-review I can press merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
Authenticate Terminus in a GitHub Actions Pipeline - Replaces session cache with encrypted session cache. It would be great to transfer this action to the Pantheon GitHub org and then publish it to the GitHub Marketplace, but okay to leave under the Lullabot org for now.
Effect
The following changes are already committed:
Remaining Work and Prerequisites
The following changes still need to be completed:
Dependencies and Timing
Release:
After date: $DATEPost Launch
Do not remove - To be completed by the docs team upon merge:
/old-path/
=>/new-path/
(if applicable)