Skip to content

fix: escape boundary labels in DFD output - #360

Open
rajanpanth wants to merge 1 commit into
OWASP:masterfrom
rajanpanth:fix/boundary-greater-than-label
Open

fix: escape boundary labels in DFD output#360
rajanpanth wants to merge 1 commit into
OWASP:masterfrom
rajanpanth:fix/boundary-greater-than-label

Conversation

@rajanpanth

Copy link
Copy Markdown

Summary

  • Escapes boundary names before inserting them into Graphviz HTML-like labels
  • Adds a regression test for a boundary name containing >

Closes #278.

Testing

  • Direct reproduction check:
    • generated a DFD with Boundary("Client > API")
    • verified output contains Client > API and not the unescaped label
  • Not run: full pytest suite is unavailable because pytest is not installed in the current Python environment.

Notes

AI assistance was used to help prepare this change.

Signed-off-by: rajanpanth <rajan.pantha@samriddhicollege.edu.np>
@rajanpanth
rajanpanth requested a review from izar as a code owner August 15, 2026 14:37
@NoodlesNZ

Copy link
Copy Markdown
Contributor

LGTM, although you need to fix your AI environment if you're going to suggest changes like this. It's trivial to get pytest working, you can run it via poetry easily. I've validated that it passes and when the Github CI kicks off, it will also validate this.

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.

Support ">" in Boundaries

2 participants