Fuzz test fixes related to pre_key_id and archived sessions count #1543
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
| name: Build and Test | |
| on: | |
| push: | |
| branches: [ main ] | |
| pull_request: # all target branches | |
| env: | |
| CARGO_TERM_COLOR: always | |
| NDK_VERSION: 25.2.9519653 | |
| RUST_BACKTRACE: 1 | |
| jobs: | |
| changes: | |
| name: Classify changes | |
| permissions: | |
| # Needed for dorny/paths-filter | |
| contents: read | |
| pull-requests: read | |
| runs-on: ubuntu-latest | |
| outputs: | |
| rust: ${{ steps.filter.outputs.rust }} | |
| java: ${{ steps.filter.outputs.java }} | |
| node: ${{ steps.filter.outputs.node }} | |
| swift: ${{ steps.filter.outputs.swift }} | |
| rust_ios: ${{ steps.filter.outputs.rust_ios }} | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - uses: dorny/paths-filter@4512585405083f25c027a35db413c2b3b9006d50 # v2.11.1 | |
| id: filter | |
| with: | |
| filters: | | |
| all: &all | |
| - '.github/workflows/build_and_test.yml' | |
| - 'bin/**' | |
| - 'rust/*' | |
| - 'rust/!(bridge|protocol)/**' | |
| - 'rust/bridge/shared/**' | |
| - 'rust/protocol/*' | |
| - 'rust/protocol/!(cross-version-testing)/**' | |
| - 'rust-toolchain' | |
| - 'Cargo.toml' | |
| - 'Cargo.lock' | |
| rust: | |
| - *all | |
| - '.clippy.toml' | |
| - '.rustfmt.license-template' | |
| - '.rustfmt.toml' | |
| - 'rust/**' # deliberately re-include rust/bridge/* and rust/protocol/cross-version-testing | |
| java: | |
| - *all | |
| - '.dockerignore' | |
| - 'java/**' | |
| - 'rust/bridge/jni/**' | |
| node: | |
| - *all | |
| - '.nvmrc' | |
| - '.prettierrc.js' | |
| - 'node/**' | |
| - 'rust/bridge/node/**' | |
| rust_ios: &rust_ios | |
| - *all | |
| - 'rust/bridge/ffi/**' | |
| swift: | |
| - *rust_ios | |
| - 'swift/**' | |
| - 'LibSignalClient.podspec' | |
| ignored: | |
| - 'LICENSE' | |
| - '*.md' | |
| - '.github/FUNDING.yml' | |
| - '.github/stale.yml' | |
| - '.github/workflows/**' | |
| - '.gitignore' | |
| - name: Check pattern completeness | |
| run: echo "::error file=.github/workflows/build_and_test.yml::File not included in any filter" && false | |
| if: ${{ !contains(steps.filter.outputs.*, 'true') }} | |
| rust: | |
| name: Rust | |
| runs-on: ubuntu-latest | |
| needs: changes | |
| if: ${{ needs.changes.outputs.rust == 'true' }} | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| version: [nightly, stable] | |
| include: | |
| - version: stable | |
| toolchain: "+stable" | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - run: sudo apt-get update && sudo apt-get install protobuf-compiler | |
| - run: rustup toolchain install $(cat rust-toolchain) --profile minimal --component rustfmt,clippy | |
| if: matrix.version == 'nightly' | |
| # This should be done before anything else | |
| # because it also checks that the lockfile is up to date. | |
| - name: Check for duplicate dependencies | |
| run: ./bin/verify_duplicate_crates | |
| if: matrix.version == 'nightly' | |
| - name: Rustfmt check | |
| run: cargo fmt --all -- --check | |
| if: matrix.version == 'nightly' | |
| - name: Rustfmt check for cross-version-testing | |
| run: cargo fmt --all -- --check | |
| working-directory: rust/protocol/cross-version-testing | |
| if: matrix.version == 'nightly' | |
| - name: Check bridge versioning | |
| run: ./bin/update_versions.py | |
| if: matrix.version == 'nightly' | |
| - name: Build | |
| run: cargo ${{ matrix.toolchain }} build --workspace --features libsignal-ffi/signal-media --verbose | |
| - name: Run tests | |
| run: cargo ${{ matrix.toolchain }} test --workspace --all-features --verbose -- --include-ignored | |
| - name: Test run benches | |
| run: cargo ${{ matrix.toolchain }} test --workspace --benches --all-features --verbose | |
| - name: Build bins and examples | |
| run: cargo ${{ matrix.toolchain }} build --workspace --bins --examples --all-features --verbose | |
| - name: Clippy | |
| run: cargo clippy --workspace --all-targets --all-features -- -D warnings | |
| if: matrix.version == 'nightly' | |
| - name: Rust docs | |
| run: cargo doc --workspace --all-features | |
| env: | |
| RUSTFLAGS: -D warnings | |
| # We check the fuzz targets on stable because they don't have lockfiles, | |
| # and crates don't generally support arbitrary nightly versions. | |
| # See https://github.com/dtolnay/proc-macro2/issues/307 for an example. | |
| - name: Check that the protocol fuzz target still builds | |
| run: cargo +stable check --all-targets | |
| working-directory: rust/protocol/fuzz | |
| env: | |
| RUSTFLAGS: --cfg fuzzing | |
| if: matrix.version == 'stable' | |
| - name: Check that the attest fuzz target still builds | |
| run: cargo +stable check --all-targets | |
| working-directory: rust/attest/fuzz | |
| env: | |
| RUSTFLAGS: --cfg fuzzing | |
| if: matrix.version == 'stable' | |
| rust32: | |
| name: Rust (32-bit testing) | |
| runs-on: ubuntu-latest | |
| needs: changes | |
| if: ${{ needs.changes.outputs.rust == 'true' }} | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| version: [nightly, stable] | |
| include: | |
| - version: stable | |
| toolchain: "+stable" | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - run: sudo apt-get update && sudo apt-get install gcc-multilib g++-multilib protobuf-compiler | |
| - run: rustup toolchain install $(cat rust-toolchain) --profile minimal --target i686-unknown-linux-gnu | |
| if: matrix.version == 'nightly' | |
| - name: Install Rust (stable) | |
| run: rustup +stable target add i686-unknown-linux-gnu | |
| if: matrix.version == 'stable' | |
| - name: Run tests (32-bit) | |
| # Exclude signal-neon-futures because those tests run Node | |
| run: cargo ${{ matrix.toolchain }} test --workspace --all-features --verbose --target i686-unknown-linux-gnu --exclude signal-neon-futures -- --include-ignored | |
| java: | |
| name: Java | |
| runs-on: ubuntu-latest | |
| needs: changes | |
| if: ${{ needs.changes.outputs.java == 'true' }} | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - name: Install NDK | |
| run: sudo ${ANDROID_HOME}/tools/bin/sdkmanager --install "ndk;${NDK_VERSION}" | |
| - run: sudo apt-get update && sudo apt-get install protobuf-compiler | |
| - run: rustup toolchain install $(cat rust-toolchain) --profile minimal --target aarch64-linux-android,armv7-linux-androideabi,x86_64-linux-android,i686-linux-android | |
| - name: Verify that the JNI bindings are up to date | |
| run: rust/bridge/jni/bin/gen_java_decl.py --verify | |
| - run: ./gradlew build assembleDebugAndroidTest | |
| working-directory: java | |
| - run: java/check_code_size.py | |
| node: | |
| name: Node | |
| runs-on: ${{ matrix.os }} | |
| strategy: | |
| matrix: | |
| os: [ubuntu-latest, windows-latest, macos-latest] | |
| needs: changes | |
| if: ${{ needs.changes.outputs.node == 'true' }} | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - run: rustup toolchain install $(cat rust-toolchain) --profile minimal | |
| # install nasm compiler for boring | |
| - name: Install nasm | |
| if: startsWith(matrix.os, 'windows') | |
| run: choco install nasm | |
| shell: cmd | |
| - run: sudo apt-get update && sudo apt-get install protobuf-compiler | |
| if: matrix.os == 'ubuntu-latest' | |
| - run: choco install protoc | |
| if: matrix.os == 'windows-latest' | |
| - run: brew install protobuf | |
| if: matrix.os == 'macos-latest' | |
| - uses: actions/setup-node@v3 | |
| with: | |
| node-version-file: '.nvmrc' | |
| - name: Verify that the Node bindings are up to date | |
| run: rust/bridge/node/bin/gen_ts_decl.py --verify | |
| if: matrix.os == 'ubuntu-latest' | |
| - run: yarn install --frozen-lockfile | |
| working-directory: node | |
| - run: yarn tsc | |
| working-directory: node | |
| - run: yarn lint | |
| if: matrix.os == 'ubuntu-latest' | |
| working-directory: node | |
| - run: yarn format -c | |
| if: matrix.os == 'ubuntu-latest' | |
| working-directory: node | |
| - run: yarn test | |
| working-directory: node | |
| swift_package: | |
| name: Swift Package | |
| runs-on: ubuntu-latest | |
| needs: changes | |
| if: ${{ needs.changes.outputs.swift == 'true' }} | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - run: rustup toolchain install $(cat rust-toolchain) --profile minimal | |
| - run: sudo apt-get update && sudo apt-get install protobuf-compiler | |
| - name: Build libsignal-ffi | |
| run: swift/build_ffi.sh -d -v --verify-ffi | |
| - name: Build Swift and run tests | |
| run: swift test -v | |
| working-directory: swift | |
| swift_cocoapod: | |
| name: Swift CocoaPod | |
| runs-on: macOS-latest | |
| needs: changes | |
| if: ${{ needs.changes.outputs.swift == 'true' }} | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - name: Run lint | |
| run: swiftlint lint --strict --reporter github-actions-logging | |
| working-directory: swift | |
| - name: Check out SignalCoreKit | |
| uses: actions/checkout@v3 | |
| with: | |
| repository: signalapp/SignalCoreKit | |
| path: SignalCoreKit | |
| - run: rustup toolchain install $(cat rust-toolchain) --profile minimal --target x86_64-apple-ios,aarch64-apple-ios-sim | |
| - run: brew install protobuf | |
| # Build only the targets that `pod lib lint` will test building. | |
| - name: Build for x86_64-apple-ios | |
| run: swift/build_ffi.sh --release | |
| env: | |
| CARGO_BUILD_TARGET: x86_64-apple-ios | |
| - name: Build for aarch64-apple-ios-sim | |
| run: swift/build_ffi.sh --release | |
| env: | |
| CARGO_BUILD_TARGET: aarch64-apple-ios-sim | |
| - name: Run pod lint | |
| # No import validation because it tries to build unsupported platforms (like 32-bit iOS). | |
| run: pod lib lint --verbose --platforms=ios --include-podspecs=SignalCoreKit/SignalCoreKit.podspec --skip-import-validation |