ScoutSuite icon indicating copy to clipboard operation
ScoutSuite copied to clipboard

Publish container to GitHub Container Registry

Open x4v13r64 opened this issue 5 years ago • 14 comments

Now that the GitHub Container Registry is in public beta (https://github.blog/2020-09-01-introducing-github-container-registry/), we could publish our new image to that registry, instead of relying on docker hub.

That would allow us to automate publishing the container and should be easier to maintain.

x4v13r64 avatar Sep 13 '20 08:09 x4v13r64

@ramimac @rossja @thommor thoughts?

x4v13r64 avatar Sep 13 '20 08:09 x4v13r64

I don't see any reason why we shouldn't do this.

I would still suggest publishing to docker hub as well though.

thommor avatar Sep 13 '20 08:09 thommor

Agreed.

On Sun, Sep 13, 2020, 04:26 Thomas Morledge [email protected] wrote:

I don't see any reason why we shouldn't do this.

I would still suggest publishing to docker hub as well though.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nccgroup/ScoutSuite/issues/855#issuecomment-691634785, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABYKFNAN65JBEJR2XHSMZTSFR63DANCNFSM4RKQY7NQ .

rossja avatar Sep 13 '20 14:09 rossja

@rossja @thommor Please check my PR

pperzyna avatar May 31 '21 10:05 pperzyna

I'm not up to scratch with GitHub actions but it looks fine to me. I'll let @rossja give their thoughts before I mark it as reviewed.

thommor avatar May 31 '21 10:05 thommor

Unfortunately, The automatically generated GITHUB_TOKEN will not work for authentication for the initial release. It will require creating the ghcr.io/nccgroup/scoutsuite:latest manual for the first time.

docker login -u ${GITHUB_NAME} -p ${GITHUB_TOKEN}  https://ghcr.io
docker push ghcr.io/nccgroup/scoutsuite:latest

pperzyna avatar May 31 '21 11:05 pperzyna

@rossja any update?

pperzyna avatar Jun 14 '21 10:06 pperzyna

sorry for the loooong delay on this. had a lot going on. working through the backlog now.

rossja avatar Aug 11 '21 20:08 rossja

@rossja ping

pperzyna avatar Mar 01 '22 15:03 pperzyna

This totally makes sense. I'll start working on it.

rossja avatar Mar 03 '22 01:03 rossja

annnnd...it's been forever. sorry. working through this now. one reason for the long delay is that i've been looking into completely restructuring the way the docker images work to significantly reduce the size of the image (right now it's ~4gb). planning to create separate images for each of the clouds (aws, azure, ibm, etc.) rather than the all-in-one that we currently use.

rossja avatar Jun 17 '22 21:06 rossja

@rossja Would it be possible to publish the version 5.12.0 to DockerHub, the current version is 5.11.0

Techbrunch avatar Sep 19 '22 12:09 Techbrunch

@Techbrunch. yep. i'll build out today. I've completely reworked how the docker stuff builds, and am ready to merge it in, but need to figure out the best way to do that because so much changed for it. (Part of that change is also to number the dockerhub images based on the version of Scoutsuite they include, btw).

rossja avatar Sep 20 '22 19:09 rossja

built and pushed

rossja avatar Sep 21 '22 19:09 rossja

@rossja How can I use the new image ? I see that you pushed base-latest 17 days ago but base was last pushed 4 month ago ?

Techbrunch avatar Oct 11 '22 09:10 Techbrunch

@Techbrunch : good catch! i had a bug in my tag script, so was naming everything into separate repositories rather than the ncc-scoutsuite one. i've fixed that now and have pushed the updates to the right spot.

rossja avatar Oct 13 '22 19:10 rossja

@rossja am I looking in the wrong place ? I'm still seeing the old version here: https://registry.hub.docker.com/layers/rossja/ncc-scoutsuite/latest/images/sha256-4455369d7eeb53bb840facd67e0adc6426d5166bcfaf2a4e378082a4e40dafa7?context=explore

Techbrunch avatar Oct 13 '22 21:10 Techbrunch

ah! yeah, that should go to ncc-scoutsuite:base-latest. i didn't update ncc-scoutsuite:latest to point to that in the tags when i restructured. i've added that to the tag script now -- but it will have the potentially unexpected side effect of NOT including aws by default (aws is in ncc-scoutsuite:aws-latest, same with the other respective clouds).

rossja avatar Oct 16 '22 11:10 rossja