From 5003f40d1d170a2b195a9d91e6acfd26cf78b9da Mon Sep 17 00:00:00 2001 From: Daniel McCarney Date: Thu, 19 Dec 2024 16:29:59 -0500 Subject: [PATCH] ci: add ech-test coverage to main CI --- .github/workflows/test.yaml | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml index 51292daf..c6e18644 100644 --- a/.github/workflows/test.yaml +++ b/.github/workflows/test.yaml @@ -127,7 +127,13 @@ jobs: if: runner.os == 'Linux' # For 'nm' run: | ! nm build/tests/client | grep '__asan_init' - ! nm build/tests/server | grep '__asan_init' + ! nm build/tests/server | grep '__asan_init' + + - name: Run ECH connect test + if: matrix.crypto == 'aws-lc-rs' # No HPKE in ring + run: | + cmake --build build --target ech-test > ech-test.log + grep 'sni=encrypted' ech-test.log # Our integration tests rely on a built-in provider being enabled. # Double-check the library/unit tests work without any providers to @@ -244,6 +250,12 @@ jobs: - name: Integration test run: cmake --build build --config "${{matrix.config}}" --target integration-test + - name: Run ECH connect test + if: matrix.crypto == 'aws-lc-rs' # No HPKE in ring + run: | + cmake --build build --target ech-test > ech-test.log + grep 'sni=encrypted' ech-test.log + ensure-header-updated: runs-on: ubuntu-latest defaults: