Skip to content

Add MobSF workflow for mobile security scanning - #288

Open
drdave-flexnetos wants to merge 3 commits into
mainfrom
FlexNetOS-patch-2-5
Open

Add MobSF workflow for mobile security scanning#288
drdave-flexnetos wants to merge 3 commits into
mainfrom
FlexNetOS-patch-2-5

Conversation

@drdave-flexnetos

Copy link
Copy Markdown
Owner

This workflow automates the MobSF security scanning process on push and pull request events to the main branch, as well as on a weekly schedule.

This workflow automates the MobSF security scanning process on push and pull request events to the main branch, as well as on a weekly schedule.
Copilot AI review requested due to automatic review settings December 6, 2025 00:12
@drdave-flexnetos
drdave-flexnetos requested review from Copilot and removed request for Copilot December 6, 2025 00:13

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 21 to 24
permissions:
contents: read # for actions/checkout to fetch code
security-events: write # for github/codeql-action/upload-sarif to upload SARIF results
actions: read # only required for a private repository by github/codeql-action/upload-sarif to get the Action run status

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Avoid SARIF upload failures on forked pull requests

This job runs on pull_request but also requests security-events: write; on forks GitHub forces the GITHUB_TOKEN to read-only, so upload-sarif will return 403 and the workflow fails without publishing MobSF results for any forked PR. Consider gating the scan to non-fork PRs or moving the SARIF upload to a trusted context (e.g., pull_request_target or a workflow_run) so fork contributions don’t constantly break the CI scan.

Useful? React with 👍 / 👎.

@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@drdave-flexnetos

Copy link
Copy Markdown
Owner Author

@cursoragent

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.

4 participants