-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fatal error: Uncaught Symfony\Component\Console\Exception\InvalidArgumentException: Incorrectly nested style tag found. in phar:///tmp/psh-go-8.0.28-4.7.1/psh-4.7.1 #85
Comments
If I set PLATFORMSH_CLI_TOKEN I get different but bad behavior:
You're welcome to experiment with this using the DDEV artifacts from ddev/ddev#5044 (comment) I think probably the golang version is not yet ready for DDEV to switch to. |
Thanks for reporting @rfay - I can't reproduce this, getting DDEV from the PR and running it on Linux, I can authenticate with all methods (except the web browser, as the port is not exported by DDEV). DDEV and Platform.sh CLI logs(.venv) akalipetis@platformify-flask-demo-web:/var/www/html$ platform
Welcome to Platform.sh!
Your projects are:
+---------------+---------------------------------+------------------+--------------------------------+
| ID | Title | Region | Organization |
+---------------+---------------------------------+------------------+--------------------------------+
[REDACTED]
+---------------+---------------------------------+------------------+--------------------------------+
Get a project by running: platform get [id]
List a project's environments by running: platform environments -p [id]
You are logged in as [REDACTED]
To view all commands, run: platform list
(.venv) akalipetis@platformify-flask-demo-web:/var/www/html$ platform version
Platform.sh CLI 4.7.1 (Wrapped) Could you please share the system you're using? Both OS and architecture should help in order to reproduce this. |
Sorry, I should have said that. This was macOS arm64/Apple Silicon, so the image is running Debian Linux/arm64. It's easy to demonstrate with just the image:
After running it and seeing the failure you can poke around in there to see what's wrong. |
Same results running on amd64 linux (Ubuntu 22.04), Docker 24.0.2 Interestingly though, doesn't happen on gitpod (also amd64 Ubuntu, but gitpod has docker 23.0.3 |
This is a bug in https://github.com/platformsh/legacy-cli The "Incorrectly nested style tag" is caused by the naive rendering of the exception, which wraps text depending on the terminal width, and incorrectly wraps style tags, and incorrectly then tries to format all of that line by line, so for example And this would have been more recently revealed by platformsh/legacy-cli@42dcd20 because it adds a couple more style tags, making it more likely to happen. So you'll get different results depending how the terminal width is set / detected. 🙃 |
In
I tried to change out the PHP CLI for the golang CLI, but I see this failure when issuing the platform command:
(
platform
cli is installed withcurl -fsSL https://raw.githubusercontent.com/platformsh/cli/main/installer.sh | bash
as you see in the PR)Is this because there aren't any projects available? Not logged in? It's certainly not expected behavior.
The text was updated successfully, but these errors were encountered: