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

Harre/net48 support and sample #113

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

hartmark
Copy link

@hartmark hartmark commented Apr 6, 2023

This library targets .net standard 2.0 so it can be used in a .net framework 4.8 project.

However if doing so you need to add your own middleware to get the headers properly set.

I have added a sample test-project that uses .net 4.8 and is wrapping CorrelationIdMiddleware as a HttpMessageHandler so we can utilize the functionality of this library.

@hartmark hartmark force-pushed the harre/net48-support-and-sample branch from 2813467 to f59eb51 Compare April 6, 2023 01:05
Also updated .net 3.1 samples to .net 6 as it is EOL
Only change to CorrelationId project is a minor nuget bump
@hartmark hartmark force-pushed the harre/net48-support-and-sample branch from f59eb51 to 7be4f3f Compare April 10, 2023 00:31
hartmark and others added 4 commits April 10, 2023 03:06
Request header is now set if a new is autogenerated
Fixed the samples with nested HttpClients
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.

1 participant