#657 Ability to set UseDefaultCredentials per route

[Sacrilege]

Closes #657

• Ocelot not working with a Windows Authentication #657

Support setting WebClient.UseDefaultCredentials per route if needed.

Docs

• Microsoft Learn: WebClient.UseDefaultCredentials Property

[clintonbale]

Would be nice to get this merged in. Very useful PR.

[Sacrilege]

What will it take to get this merged in to main?
@TomPallister

[raman-m]

@Sacrilege Hi Troy!
Could you resolve merge conflicts please?
And what issue is this PR related to?

[raman-m]

The feature branch has been rebased onto ThreeMammals:develop successfully!
Welcome to code review!

[raman-m]

@Sacrilege commented on Aug 8, 2022

I hope, merging will happen soon... 🤣

[raman-m]

@ggnaegi commented on April 3:

Mocking UseDefaultCredentials in an acceptance test on Linux hosts could be quite a pain...

Indeed, it can be challenging. However, we could set up a Windows build machine as a Docker image and direct CircleCI to execute the Windows build by downloading the image with the Windows OS from Docker Hub.
Have you encountered similar issues in your development?

Roadmap

• Write all tests and verify them locally using the Visual Studio IDE on Windows OS. Ensure all tests pass.
• Conduct initial builds on CircleCI using our existing ocelot2/circleci-build image with Linux OS aka Alpine 3.xx. It's likely that some tests will fail.
• Develop a new Docker image based on Windows OS and upload it to the ocelot2 account. All three .NET SDKs must be preinstalled.
• Modify the existing build.cake script and/or CircleCI configuration in the repository to detect Windows build scenarios and dynamically create the necessary Windows OS image container. This may present a DevOps challenge, but CircleCI workflows should offer some functionality to select the desired Docker image based on a condition.
• Achieve a successful build on CircleCI with all checks passing.

Links

• Official Docker images for the .NET SDK
• Ocelot Gateway | Docker Hub
• Using the Windows execution environment | CircleCI Docs

[raman-m]

TODO

• Expand unit testing, particularly for the MessageInvokerPool class
• Conduct acceptance testing, including trials on a Windows machine

[ggnaegi]

TODO

• Expand unit testing, particularly for the MessageInvokerPool class
• Conduct acceptance testing, including trials on a Windows machine

So, you're expecting that quality level from our side and you merged some stuff this week out of the blue...

Ok fine then.

[raman-m]

@ggnaegi Ups! Yet another rebase is needed! 😄
You want this feature, right? You're assigned to have some pair programming fun.

[raman-m]

New Docker images based on Windows OS

• OS/Arch: windows/amd64

Testing of Dev SSL certificates is required!