-
Notifications
You must be signed in to change notification settings - Fork 5
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
php-composer.yml: install ability with/without validating certificate #105
base: master
Are you sure you want to change the base?
php-composer.yml: install ability with/without validating certificate #105
Conversation
@DavidHume-ArtefactualSystems : I was wondering if the failure is related/similar to e.g. composer/composer#5685 , composer/composer#7317 (curl failing to retrieve https://composer.github.io/installer.sig ) The workaround mentioned in these issues is |
Thanks @hakamine - I thought I checked out that aspect. This is happening when I deploy to new RBCM servers when on their VPN (I take it you haven't run into similar with SFU, CVA). The ca-certificates presence and version seems to be okay
seems to compare favourably to atomeurope-nginx, where we don't have a problem deploying
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (but I have not been able to test)
Thanks Hector. It is interesting - not quite the same problem as the composer issues you listed, but ran similar test as mentioned there and first few lines suggest validating certificates present
plus Miguel did a wget test proving could reach installer.sig... and yet when I initially deployed to new RBCM machines, TASK [artefactual.atom : Get Composer installer signature.] ******** fails with
hence my thought to make validate_certs=False configurable. ¯_(ツ)_/¯ |
Hi @hakamine , @mamedin - so I did try a test while deploying RBCM's new public server, failed right away because the syntax was wrong in that context. However, while the new syntax (see the new "Files changed") did run, for atom_php_composer_validate_certs defined, got a failure on the subsequent task
whereas when the new variable is undefined, proceeded as normal (same as current role). Unlike my deploy to RbcmU18AtomProdWeb, the deploy to RbcmU18AtomPubWeb didn't NEED validate_certs: "no" , so carried on. Still think this option is a good idea, but will have to come back to it. |
If need to include validate_certs: no can do by ansible variable atom_php_composer_validate_certs: "no"