Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Patch release: Don't reveal auth tokens #16

Merged
merged 1 commit into from
Sep 6, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# nf-core/tower-action: Changelog

## [[v3.2.1](https://github.com/nf-core/tower-action/releases/tag/v3.2.1)] - 2022-09-06

* Patch release to strip Tower authentication tokens from the log file 😬

## [[v3.2.0](https://github.com/nf-core/tower-action/releases/tag/v3.2.0)] - 2022-09-06

* Bumped tower-cli version `0.6.2` -> `0.6.4`
Expand Down
7 changes: 7 additions & 0 deletions entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@
# Use `tee` to print just stdout to the console but save stdout + stderr to a file
LOG_FN="tower_action_"$(date +'%Y_%m_%d-%H_%M')".log"

# Manual curl of service-info
curl https://api.tower.nf/service-info >> $LOG_FN
echo -e "\n\n------\n\n" >> $LOG_FN

# Health check - print basic settings
tw -v info \
2>> $LOG_FN | tee -a $LOG_FN
Expand All @@ -28,3 +32,6 @@ tw -v \
${PRE_RUN_SCRIPT:+"--pre-run=pre_run.sh"} \
${NEXTFLOW_CONFIG:+"--config=nextflow.config"} \
2>> $LOG_FN | tee -a $LOG_FN

# Strip secrets from the log file
sed -i "s/$TOWER_ACCESS_TOKEN/xxxxxx/" $LOG_FN