Skip to content

Draft: SIDI-123 - Improve yarn audits#29

Open
odemolliens wants to merge 4 commits intodevelopfrom
feat/SIDI-123/yarn_audit_improvements
Open

Draft: SIDI-123 - Improve yarn audits#29
odemolliens wants to merge 4 commits intodevelopfrom
feat/SIDI-123/yarn_audit_improvements

Conversation

@odemolliens
Copy link
Owner

@odemolliens odemolliens commented Feb 26, 2025

Screenshot 2025-02-26 at 10 58 25
Screenshot 2025-02-26 at 10 58 59

@odemolliens
Copy link
Owner Author

@imranMnts must be adapted for Bitrise

# hack to fix yarn version issue (temporary!)

# 🚀 Hack to fix yarn version issue (temporary!)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it still required?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think so

CRITICAL_COUNT=$(jq -r '.total.critical' $AGGREGATED_JSON)

# Export vulnerability count for further use
echo "LOW_COUNT=$LOW_COUNT" #>> $CM_ENV
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@imranMnts here we need adapt for bitrise

MODERATE_COUNT=${BASH_REMATCH[1]}
fi
# 📂 Define the export directory for all reports
export CM_EXPORT_DIR="build/reports"
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@imranMnts here we need adapt for bitrise

sendToTeams "$webhook_url" "$html_content"
}

sendVulnerabilityReportToTeams "$TEAMS_SECURITY_WEBHOOCKS"
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@odemolliens check if working if no $TEAMS_SECURITY_WEBHOOCKS provided

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants