Skip to content

Commit

Permalink
ci: address OSS scorecard warnings
Browse files Browse the repository at this point in the history
* remediate the `Token-Permissions` warning/low score by moving the F5 CLA GH Action permissions around
* remediate the `Pinned-Dependencies` warning/low score by using specific Docker image SHAs and adding Docker image updates to Dependabot
* hopefully remediate the `Vulnerabilities` warning/low score by recreating `package-lock.json`
  • Loading branch information
alessfg committed May 6, 2024
1 parent 62caea4 commit 076daf9
Show file tree
Hide file tree
Showing 12 changed files with 342 additions and 248 deletions.
6 changes: 6 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,12 @@
---
version: 2
updates:
- package-ecosystem: docker
directory: /
schedule:
interval: weekly
day: monday
time: "00:00"
- package-ecosystem: github-actions
directory: /
schedule:
Expand Down
13 changes: 7 additions & 6 deletions .github/workflows/f5-cla.yml
Original file line number Diff line number Diff line change
@@ -1,18 +1,19 @@
---
name: F5 CLA
on:
issue_comment:
types: [created]
pull_request_target:
types: [opened, closed, synchronize]

permissions:
actions: write
pull-requests: write
statuses: write

permissions: read-all
jobs:
f5-cla:
name: F5 CLA
runs-on: ubuntu-22.04
permissions:
actions: write
pull-requests: write
statuses: write
steps:
- name: Run F5 Contributor License Agreement (CLA) assistant
if: (github.event.comment.body == 'recheck' || github.event.comment.body == 'I have hereby read the F5 CLA and agree to its terms') || github.event_name == 'pull_request_target'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
name: CI
name: NGINX S3 Gateway CI/CD
on:
push:
branches: [main]
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.buildkit.plus
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM debian:bookworm-slim
FROM debian:bookworm-slim@sha256:155280b00ee0133250f7159b567a07d7cd03b1645714c3a7458b2287b0ca83cb

ENV NGINX_PLUS_VERSION 30-2
ENV NGINX_VERSION 1.25.1
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.latest-njs
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# This container image removes the existing njs package from the inherited image
# (which could be OSS NGINX or NGINX Plus), builds njs from the latest
# source, and installs it.
FROM nginx-s3-gateway
FROM nginxinc/nginx-s3-gateway@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4

RUN set -eux \
export DEBIAN_FRONTEND=noninteractive; \
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.oss
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM nginx:1.25.3
FROM nginx:1.25.5@sha256:711cc227e3c4181ab27cde13cf662f6dd1d06d16b3344f871c6d04cbff22f6f8

ENV NGINX_VERSION "1.25.3"
ENV NJS_VERSION "0.8.2"
Expand Down
4 changes: 2 additions & 2 deletions Dockerfile.plus
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
FROM debian:bookworm-slim
FROM debian:bookworm-slim@sha256:155280b00ee0133250f7159b567a07d7cd03b1645714c3a7458b2287b0ca83cb

ENV NGINX_PLUS_VERSION 30-2
ENV NGINX_VERSION 1.25.1
ENV NJS_VERSION 30+0.8.0-1
ENV XSLT_VERSION 30-1

ENV PROXY_CACHE_MAX_SIZE "10g"
ENV PROXY_CACHE_INACTIVE "60m"
ENV PROXY_CACHE_SLICE_SIZE "1m"
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.unprivileged
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# to allow running NGINX S3 Gateway as a non root user.
# Steps are based on the official unprivileged container:
# https://github.com/nginxinc/docker-nginx-unprivileged/blob/main/Dockerfile-debian.template
FROM nginx-s3-gateway
FROM nginxinc/nginx-s3-gateway@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4

# Implement changes required to run NGINX as an unprivileged user
RUN sed -i "/^server {/a \ listen 8080;" /etc/nginx/templates/default.conf.template \
Expand Down
2 changes: 1 addition & 1 deletion examples/brotli-compression/Dockerfile.oss
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM nginxinc/nginx-s3-gateway
FROM nginxinc/nginx-s3-gateway@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4

ENV BROTLI_VERSION "v1.0.0rc"

Expand Down
2 changes: 1 addition & 1 deletion examples/gzip-compression/Dockerfile.oss
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
FROM nginxinc/nginx-s3-gateway
FROM nginxinc/nginx-s3-gateway@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4

COPY etc/nginx/conf.d /etc/nginx/conf.d
2 changes: 1 addition & 1 deletion examples/modsecurity/Dockerfile.oss
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM nginxinc/nginx-s3-gateway
FROM nginxinc/nginx-s3-gateway@sha256:8aa48324479b3653b5936183cc97f2ca1aa9078d229042f1bca357834bd906f4

ENV MODSECURITY_VERSION "v1.0.1"
ENV OWASP_RULESET_VERSION "v3.3.0"
Expand Down
Loading

0 comments on commit 076daf9

Please sign in to comment.