Ensure notification orders are placed within transaction - Githubissues

Altinn / altinn-notifications

Altinn platform microservice for handling notifications (mail, sms, etc)

MIT License

3 stars 3 forks source link

Ensure notification orders are placed within transaction #475

Closed martinothamar closed 3 months ago

martinothamar commented 5 months ago

Description

I noticed that orders are not included in the transaction when executed. When CreateCommand is invoked on NpgsqlDataSource, it is creating a new connection

I also awaited each secondary insert as well, as the connection itself is not threadsafe

Related Issue(s)

N/A

Verification

[x] Your code builds clean without any errors or warnings
[x] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)