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.

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

Modifying the ObjectTrace logger to output a single message for a bundle instead of a message for each session #627

Merged

Conversation

arp-0984
Copy link
Collaborator

@arp-0984 arp-0984 commented Nov 27, 2023

Overloading the emitLifecycleEvent method because we do not yet have a IBDO object where we need to emit this message and we do not need it to output the fields we need. The old emitLifecycleEvent can probably be removed once this is fully integrated.

@cfkoehler cfkoehler self-requested a review November 27, 2023 17:16
Copy link
Collaborator

@cfkoehler cfkoehler left a comment

Choose a reason for hiding this comment

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

I integrated this with the upstream project and got single output per file which is expected

@jpdahlke jpdahlke modified the milestones: v8.0.0-M12, v8.0.0-M11 Dec 14, 2023
@cfkoehler cfkoehler added enhancement An enhancement to an existing feature integration Effort is required from downstream customers and removed integration Effort is required from downstream customers labels Dec 15, 2023
@jpdahlke jpdahlke merged commit 2664bb3 into NationalSecurityAgency:master Dec 15, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants