Closed samuelbernardolip closed 4 years ago
@orviz There is a bug since credentials variable appears duplicated. We need to solve that before doing the merge. This PR already as the changes from test/credentials-into-docker-compose, so this branch can also be deleted in the end.
just removed test/credentials-into-docker-compose
branch
There was an extra call to credentialsToStep()
method in a DEBUG line. The last commit just comments this line out.
I've tried to store the return value of credentialsToStep()
so to be used both for the debugging and for the withCredentials
statement, but when I do that I get a complain from Jenkins script security (org.jenkinsci.plugins.scriptsecurity.sandbox.RejectedAccessException: Scripts not permitted to use method groovy.lang.GroovyObject setProperty java.lang.String java.lang.Object
)
Pass credentials into docker compose using arguments provided to exec comand. resolves PR #65 PR #82