Skip to content

Commit c46b314

Browse files
authored
Try fixing exodus CI job (#8721)
* Ignore tracker count retrieved by exodus job so the report can be created * Ignore Sentry, since it's a known and optional tracker
1 parent dff9b82 commit c46b314

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

.github/workflows/build.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,12 +79,18 @@ jobs:
7979
- name: Execute exodus-standalone
8080
uses: docker://exodusprivacy/exodus-standalone:latest
8181
with:
82-
args: /github/workspace/gplay/release/vector-gplay-universal-release-unsigned.apk -j -o /github/workspace/exodus.json
82+
# Don't fail when finding trackers so they can be reported later
83+
args: /github/workspace/gplay/release/vector-gplay-universal-release-unsigned.apk -j -o /github/workspace/exodus.json -e 0
8384
- name: Upload exodus json report
8485
uses: actions/upload-artifact@v3
8586
with:
8687
name: exodus.json
8788
path: |
8889
exodus.json
8990
- name: Check for trackers
90-
run: "jq -e '.trackers == []' exodus.json > /dev/null || { echo '::error static analysis identified user tracking library' ; exit 1; }"
91+
env:
92+
SENTRY_ID: 447
93+
# Should only contain a Sentry item
94+
run: |
95+
TRACKER_IDS=$(jq ".trackers[] | .id" exodus.json)
96+
[ $TRACKER_IDS = ${{ env.SENTRY_ID }} ] || { echo '::error static analysis identified user tracking library' ; exit 1; }

0 commit comments

Comments
 (0)