-
Notifications
You must be signed in to change notification settings - Fork 8
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
AIO app undeploy throws an error on Windows terminal (Powershell path invalid) #80
Comments
Output of aio info: Proxies: |
Issue occurs with various aio commands, e.g. also |
JIRA issue created: https://jira.corp.adobe.com/browse/ACNA-2336 |
Root CauseIssue is not in the actual aio commands, but the fact that login is required at some point. WoraroundWhen error occurs run |
Doing a separate login command is just a workaround, not a solution. The env var is definitely the problem since as you can see from the log there is an undefined in We use open aio-lib-ims-oauth/src/login.js Line 63 in c0ea016
aio-lib-ims-oauth/package.json Line 11 in c0ea016
and this is where it is failing: https://github.com/sindresorhus/open/blob/cbc008bab21f657475b54e33a823b2941737da6f/index.js#L147 This is probably the actual issue sindresorhus/open#292 Might have to do this patch/workaround. It's probably an underlying node issue sanity-io/sanity#4221 |
PR #81 |
Expected Behaviour
aio app undeploy
runs successfully from Windows CMD terminal and undeploys the AB appActual Behaviour
aio app undeploy
throws an error in latest app plugin 10.2.2. Apparently this is because the Windows root path isundefined
. The Powershell is not used itself however to run the undeploy command.Steps to Reproduce
Run
aio app undeploy
on Windows in CMD terminalEnvironment Info
Reference
https://adobe-acs.slack.com/archives/C013UDBFBGB/p1686579180629279
The text was updated successfully, but these errors were encountered: