Skip to content
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

Updated actions for docker hub to build and publish develop version #73

Merged
merged 3 commits into from
Sep 17, 2020
Merged

Updated actions for docker hub to build and publish develop version #73

merged 3 commits into from
Sep 17, 2020

Conversation

eugene-manuilov
Copy link
Collaborator

@eugene-manuilov eugene-manuilov commented Sep 16, 2020

Description of the Change

Reworked GitHub Actions workflow for docker hub images to support docker images for develop branch.

Alternate Designs

Benefits

Possible Drawbacks

Verification Process

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Applicable Issues

Changelog Entry

@helen
Copy link

helen commented Sep 17, 2020

I noticed in your other PR that you ended up moving away from Docker's actions - is there any reason why? I'm wary of echoing out the access token like this, GH does its best to mask any it detects from log output but still seems like an unnecessary risk if we can make it work with actions that take them as inputs.

@eugene-manuilov
Copy link
Collaborator Author

I noticed in your other PR that you ended up moving away from Docker's actions - is there any reason why?

Which PR do you mean, @helen? Can you link it?

I'm wary of echoing out the access token like this, GH does its best to mask any it detects from log output but still seems like an unnecessary risk if we can make it work with actions that take them as inputs.

It's a good point. I have updated it to use secrets as environment variables which seems to be sufficient based on the GitHub Actions documentation.

@tlovett1
Copy link
Member

As long as we are confident this isn't printing secrets to the logs, I'm good to merge.

@eugene-manuilov
Copy link
Collaborator Author

eugene-manuilov commented Sep 17, 2020

I have added dummy secrets to my fork and can confirm that GitHub doesn't expose it when I check the log:

login2

@tlovett1 tlovett1 merged commit 18c73c2 into 10up:develop Sep 17, 2020
@jeffpaul jeffpaul added this to the 2.1.0 milestone Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants