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

Extracting data from payload and adding them as amqp properties #600

Conversation

thorhalvor
Copy link
Contributor

@thorhalvor thorhalvor commented Dec 5, 2023

Feature added to improve auditing and tracing
Important none-sensitive properties are extracted from the payload and added as amqp application properties.
This makes tracing MessageIds alot easier and also open for correlation between businessdocument and application receipt

@thorhalvor
Copy link
Contributor Author

Data extracted for the different type of message:

Hodemelding Apprec Other
X-MessageInfo-Type * *
X-MessageInfo-MsgId * *
X-MessageInfo-RefToParent *
X-MessageInfo-RefToConversation *
X-MessageInfo-SchemaNamespace * * *
X-MessageInfo-OriginalMsgId *
X-MessageInfo-OriginalMsgType *
X-Sender-Lvl1Id * *
X-Sender-Lvl2Id * *
X-Receiver-Lvl1Id * *
X-Receiver-Lvl2Id * *
X-AttachmentInfo-Count * *
X-AttachmentInfo-TotalSizeInBytes * *
X-AttachmentInfo-HasExternalReference * *

@thorhalvor thorhalvor changed the title extracting data from payload and addiing them to amqp properties Extracting data from payload and adding them as amqp properties Dec 6, 2023
adding minor doc summary

refactor based on pr comments
…om:thorhalvor/Helsenorge.Messaging into feature/add-payload-metadata-to-properties
@MartinTGD
Copy link
Collaborator

please fix the last warnings and review my comment. Other than that it looks good

adding summary to MetadtaKeys class
@MartinTGD MartinTGD merged commit 8a47ddb into helsenorge:master Dec 11, 2023
4 checks passed
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.

2 participants