Skip to content

Comments

Use Multi-Stage-Build For Nikto#642

Merged
nigthknight merged 3 commits intomainfrom
nikto-custom-dockerfile
Sep 14, 2021
Merged

Use Multi-Stage-Build For Nikto#642
nigthknight merged 3 commits intomainfrom
nikto-custom-dockerfile

Conversation

@nigthknight
Copy link
Member

@nigthknight nigthknight commented Sep 13, 2021

If applied, this PR will:

Signed-off-by: Yannik Fuhrmeister [email protected]

@nigthknight nigthknight added ci Changes to the continuous integration setup enhancement New feature or request maintenance scanner Implement or update a security scanner labels Sep 13, 2021
@nigthknight nigthknight self-assigned this Sep 13, 2021
@nigthknight nigthknight marked this pull request as draft September 13, 2021 10:39
@nigthknight nigthknight force-pushed the nikto-custom-dockerfile branch 5 times, most recently from 54a4575 to 90f1213 Compare September 13, 2021 11:54
@nigthknight nigthknight marked this pull request as ready for review September 13, 2021 11:57
SebieF
SebieF previously approved these changes Sep 13, 2021
As described in #610 the build for nikto was quite complicated and could
not be managed via matrix builds on GH.

To Fix this I created a multi-stage Dockerfile that clones the nikto
repo in the build stage and checks out the specified version.

The next stage installs all dependencies like in the Dockerfile provided
by the maintainer of nikto, copies nikto from the build stage and our
wrapper script.

I also noticed that the alpine version was a little behind so I changed
it from 3.12 to 3.14

Signed-off-by: Yannik Fuhrmeister <[email protected]>
With the new multi-stage image it should be possible to add nikto to the
matrix build for 3rd-party scanners.

So I removed the build job for nikto and added nikto to the matrix

Signed-off-by: Yannik Fuhrmeister <[email protected]>
@nigthknight nigthknight force-pushed the nikto-custom-dockerfile branch from 2633c1b to 5be6d7b Compare September 13, 2021 12:58
Signed-off-by: Yannik Fuhrmeister <[email protected]>
@nigthknight nigthknight merged commit b4df5f4 into main Sep 14, 2021
@nigthknight nigthknight deleted the nikto-custom-dockerfile branch September 14, 2021 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes to the continuous integration setup enhancement New feature or request maintenance scanner Implement or update a security scanner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🏗 Migrate Nikto Build to be in line with other 3rd party scanners

3 participants