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

Updates the location the binding.pem is downloaded to for Lockr support #2859

Closed
wants to merge 2 commits into from

Conversation

cellar-door
Copy link

Small update to point the download of the binding.pem to the correct folder where Lockr can find it. This is related to https://github.com/lando/lando/issues/2855 in that without the PANTHEON_BINDING not being defined Lockr ceases to work, however we are moving off using that constant and this will be the fix for that issue.

@nullvariable
Copy link
Contributor

this looks good to me

@pirog
Copy link
Member

pirog commented Feb 23, 2021

@cellar-door @nullvariable im going to close this in favor of #2876 which I THINK will get us where we want to go. If i'm wrong about that i'll owe you both a beer 🍺

@pirog pirog closed this Feb 23, 2021
pirog added a commit that referenced this pull request Feb 23, 2021
#2859: Allow access to pantheon binding.pem from new location
pirog added a commit that referenced this pull request Feb 23, 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