reverseproxy: only change the content-length header when the full request was buffered #4348
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| # Used as inspiration: https://github.com/mvdan/github-actions-golang | |
| name: Tests | |
| on: | |
| push: | |
| branches: | |
| - master | |
| - 2.* | |
| pull_request: | |
| branches: | |
| - master | |
| - 2.* | |
| jobs: | |
| test: | |
| strategy: | |
| # Default is true, cancels jobs for other platforms in the matrix if one fails | |
| fail-fast: false | |
| matrix: | |
| os: | |
| - ubuntu-latest | |
| - macos-latest | |
| - windows-latest | |
| go: | |
| - '1.20' | |
| - '1.21' | |
| include: | |
| # Set the minimum Go patch version for the given Go minor | |
| # Usable via ${{ matrix.GO_SEMVER }} | |
| - go: '1.20' | |
| GO_SEMVER: '~1.20.6' | |
| - go: '1.21' | |
| GO_SEMVER: '~1.21.0' | |
| # Set some variables per OS, usable via ${{ matrix.VAR }} | |
| # CADDY_BIN_PATH: the path to the compiled Caddy binary, for artifact publishing | |
| # SUCCESS: the typical value for $? per OS (Windows/pwsh returns 'True') | |
| - os: ubuntu-latest | |
| CADDY_BIN_PATH: ./cmd/caddy/caddy | |
| SUCCESS: 0 | |
| - os: macos-latest | |
| CADDY_BIN_PATH: ./cmd/caddy/caddy | |
| SUCCESS: 0 | |
| - os: windows-latest | |
| CADDY_BIN_PATH: ./cmd/caddy/caddy.exe | |
| SUCCESS: 'True' | |
| runs-on: ${{ matrix.os }} | |
| steps: | |
| - name: Checkout code | |
| uses: actions/checkout@v4 | |
| - name: Install Go | |
| uses: actions/setup-go@v4 | |
| with: | |
| go-version: ${{ matrix.GO_SEMVER }} | |
| check-latest: true | |
| # These tools would be useful if we later decide to reinvestigate | |
| # publishing test/coverage reports to some tool for easier consumption | |
| # - name: Install test and coverage analysis tools | |
| # run: | | |
| # go get github.com/axw/gocov/gocov | |
| # go get github.com/AlekSi/gocov-xml | |
| # go get -u github.com/jstemmer/go-junit-report | |
| # echo "$(go env GOPATH)/bin" >> $GITHUB_PATH | |
| - name: Print Go version and environment | |
| id: vars | |
| shell: bash | |
| run: | | |
| printf "Using go at: $(which go)\n" | |
| printf "Go version: $(go version)\n" | |
| printf "\n\nGo environment:\n\n" | |
| go env | |
| printf "\n\nSystem environment:\n\n" | |
| env | |
| printf "Git version: $(git version)\n\n" | |
| # Calculate the short SHA1 hash of the git commit | |
| echo "short_sha=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT | |
| - name: Get dependencies | |
| run: | | |
| go get -v -t -d ./... | |
| # mkdir test-results | |
| - name: Build Caddy | |
| working-directory: ./cmd/caddy | |
| env: | |
| CGO_ENABLED: 0 | |
| run: | | |
| go build -trimpath -ldflags="-w -s" -v | |
| - name: Publish Build Artifact | |
| uses: actions/upload-artifact@v3 | |
| with: | |
| name: caddy_${{ runner.os }}_go${{ matrix.go }}_${{ steps.vars.outputs.short_sha }} | |
| path: ${{ matrix.CADDY_BIN_PATH }} | |
| # Commented bits below were useful to allow the job to continue | |
| # even if the tests fail, so we can publish the report separately | |
| # For info about set-output, see https://stackoverflow.com/questions/57850553/github-actions-check-steps-status | |
| - name: Run tests | |
| # id: step_test | |
| # continue-on-error: true | |
| run: | | |
| # (go test -v -coverprofile=cover-profile.out -race ./... 2>&1) > test-results/test-result.out | |
| go test -v -coverprofile="cover-profile.out" -short -race ./... | |
| # echo "status=$?" >> $GITHUB_OUTPUT | |
| # Relevant step if we reinvestigate publishing test/coverage reports | |
| # - name: Prepare coverage reports | |
| # run: | | |
| # mkdir coverage | |
| # gocov convert cover-profile.out > coverage/coverage.json | |
| # # Because Windows doesn't work with input redirection like *nix, but output redirection works. | |
| # (cat ./coverage/coverage.json | gocov-xml) > coverage/coverage.xml | |
| # To return the correct result even though we set 'continue-on-error: true' | |
| # - name: Coerce correct build result | |
| # if: matrix.os != 'windows-latest' && steps.step_test.outputs.status != ${{ matrix.SUCCESS }} | |
| # run: | | |
| # echo "step_test ${{ steps.step_test.outputs.status }}\n" | |
| # exit 1 | |
| s390x-test: | |
| name: test (s390x on IBM Z) | |
| runs-on: ubuntu-latest | |
| if: github.event.pull_request.head.repo.full_name == github.repository && github.actor != 'dependabot[bot]' | |
| continue-on-error: true # August 2020: s390x VM is down due to weather and power issues | |
| steps: | |
| - name: Checkout code | |
| uses: actions/checkout@v4 | |
| - name: Run Tests | |
| run: | | |
| mkdir -p ~/.ssh && echo -e "${SSH_KEY//_/\\n}" > ~/.ssh/id_ecdsa && chmod og-rwx ~/.ssh/id_ecdsa | |
| # short sha is enough? | |
| short_sha=$(git rev-parse --short HEAD) | |
| # The environment is fresh, so there's no point in keeping accepting and adding the key. | |
| rsync -arz -e "ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null" --progress --delete --exclude '.git' . "$CI_USER"@ci-s390x.caddyserver.com:/var/tmp/"$short_sha" | |
| ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -t "$CI_USER"@ci-s390x.caddyserver.com "cd /var/tmp/$short_sha; go version; go env; printf "\n\n";CGO_ENABLED=0 go test -v ./..." | |
| test_result=$? | |
| # There's no need leaving the files around | |
| ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null "$CI_USER"@ci-s390x.caddyserver.com "rm -rf /var/tmp/'$short_sha'" | |
| echo "Test exit code: $test_result" | |
| exit $test_result | |
| env: | |
| SSH_KEY: ${{ secrets.S390X_SSH_KEY }} | |
| CI_USER: ${{ secrets.CI_USER }} | |
| goreleaser-check: | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Checkout code | |
| uses: actions/checkout@v4 | |
| - uses: goreleaser/goreleaser-action@v5 | |
| with: | |
| version: latest | |
| args: check | |
| env: | |
| TAG: ${{ steps.vars.outputs.version_tag }} |