Skip to content

fix: allow parsing of github expressions containing non-breaking-space characters, and allow dataflow analysis to fail#1340

Merged
nicallen merged 2 commits intomainfrom
nicallen/fix-dataflow-analysis-crashes
Mar 30, 2026
Merged

fix: allow parsing of github expressions containing non-breaking-space characters, and allow dataflow analysis to fail#1340
nicallen merged 2 commits intomainfrom
nicallen/fix-dataflow-analysis-crashes

Conversation

@nicallen
Copy link
Copy Markdown
Member

@nicallen nicallen commented Mar 30, 2026

…to analyse a workflow without crashing Macaron entirely.

Summary

Fixes a crash caused by github expression parser not handling non-breaking-space characters (encountered when analysing https://github.com/openjdk/lanai and https://github.com/openjdk/jdk17u-ri).
Also allows Macaron to continue even if dataflow analysis fails (in which case only analysis of the offending workflow is skipped), which is useful to mitigate the impact of analysis bugs, as well as gracefully handle cases where a workflow is genuinely malformed (e.g. in one of the workflows of pkg:pypi/jupyter-client@8.7.0).
Also improves error reporting when the dataflow analysis fails, to report the failing workflow file.

Description of changes

Related issues

Checklist

  • I have reviewed the contribution guide.
  • My PR title and commits follow the Conventional Commits convention.
  • My commits include the "Signed-off-by" line.
  • I have signed my commits following the instructions provided by GitHub. Note that we run GitHub's commit verification tool to check the commit signatures. A green verified label should appear next to all of your commits on GitHub.
  • I have updated the relevant documentation, if applicable.
  • I have tested my changes and verified they work as expected.

…e characters, and allow dataflow analysis to fail to analyse a workflow without crashing Macaron entirely.

Signed-off-by: Nicholas Allen <nicholas.allen@oracle.com>
@nicallen nicallen requested a review from behnazh-w as a code owner March 30, 2026 01:55
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Mar 30, 2026
Signed-off-by: Nicholas Allen <nicholas.allen@oracle.com>
@behnazh-w behnazh-w changed the title fix: allow parsing of github expressions containing non-breaking-spac… fix: allow parsing of github expressions containing non-breaking-space characters, and allow dataflow analysis to fail Mar 30, 2026
@nicallen nicallen merged commit 011e21a into main Mar 30, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants