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

Look for presence of acli_key in cloud_api.conf #19

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

joegl
Copy link
Contributor

@joegl joegl commented Mar 3, 2025

blt-sws currently expects cloud_api.conf to be formatted like:

{"key":"ACQUIA KEY HERE","secret":"ACQUIA SECRET HERE"}

ACLI re-writes cloud_api.conf to look like:

array(4) {
  ["keys"]=>
  object(stdClass)#1050 (1) {
    ["ACQUIA KEY HERE"]=>
    object(stdClass)#1057 (3) {
      ["secret"]=>
      string(44) "ACQUIA SECRET HERE"
      ["uuid"]=>
      string(36) "UUID HERE"
    }
  }
  ["acli_key"]=>
  string(36) "ACQUIA KEY HERE"
}

This attempts to add support for the ACLI formatted way.

@joegl joegl requested a review from pookmish March 3, 2025 17:56
@joegl joegl requested a review from pookmish March 3, 2025 19:54
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.

2 participants