Replies: 5 comments 16 replies
-
|
You can disable it using the undocumented I think users must be warned about this behaviour at least on the getting started doc page. |
Beta Was this translation helpful? Give feedback.
-
|
How about removing show banner, which has nothing to do with deployer. It even gets the content from some random other domain. You could even call it a security risk, if for some reason you lose control over that domain. Requesting and printing random data from random domains is a no go. I have seen the env variable, it is not a solution. Seems more like it was put there only to enable you to say: "oh yeah but you can disable it". Whichs is exactly what happened. Nobody wants to define variables when calling dep. |
Beta Was this translation helpful? Give feedback.
-
I see you request. Okay. |
Beta Was this translation helpful? Give feedback.
-
|
This also introduces new default behavior that breaks using the output of a task in a script. For example, I have a I use the output of that task to pipe to another script to send myself a Slack message. The banner change breaks that. Mine is a trivial example, and the fix add the env var or use FWIW I will echo what others said here about this being an erosion of trust. I've used deployer for years and have even trained dev teams on how to use it. Yes, like mentioned above, I should have seen the banner addition to the source code years ago. Still, waking up this morning and having to work around an advertisement to deploy my app has really tainted my trust in deployer. I do truly appreciate everything you've done for the community. You are free to do anything you want with the project, of course. I hope you will reconsider your approach to this so that users do not need to worry about undocumented behavior and HTTP calls in a mission critical tool that deals with sensitive data. |
Beta Was this translation helpful? Give feedback.
-
|
I removed the banner. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Deployer Version
7.5.12
Target OS
Does not matter
Which PHP version are you using?
PHP 8.4
Content of deploy.php or deploy.yaml
No response
Steps to reproduce
Deploy something, and you get
Beta Was this translation helpful? Give feedback.
All reactions