Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix Mark and SecMark Attribute Parsing #3

Merged
merged 2 commits into from
Nov 16, 2023
Merged

Fix Mark and SecMark Attribute Parsing #3

merged 2 commits into from
Nov 16, 2023

Conversation

mbolt35
Copy link
Contributor

@mbolt35 mbolt35 commented Nov 16, 2023

Issue: SecMark was being applied to mark on the Flow model incorrectly.
Fix:

  • Added a sec_mark field to Flow and correctly set on SecMark attribute
  • Added handling for Mark attribute and correctly set mark on Flow

cliffcolvin
cliffcolvin reviewed Nov 16, 2023
View reviewed changes
Copy link

@cliffcolvin cliffcolvin left a comment

Choose a reason for hiding this comment

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

TY!

@mbolt35 mbolt35 merged commit 5acb73c into main Nov 16, 2023
8 checks passed
@mbolt35 mbolt35 deleted the bolt/cta-mark branch November 16, 2023 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cliffcolvin cliffcolvin cliffcolvin left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mbolt35 @cliffcolvin