Matt Connew

LGTM. Can you rebase and squash your changes into a single commit then I'll merge.

`SecurityMode.Message` is not supported. If you control the client and the server, you could switch to using `SecurityMode.TransportWithMessageCredentials` and the authentication flow/configuration will continue to work exactly as before. You...

This could be a problem in the client or the service side, or it could be both. Can you help work out which it is by running .NET Framework client...

I just noticed you are using a sync contract. If you are using the .NET WCF Client nuget packages, they don't actually have a synchronous implementation, it does sync over...

We 100% agree that being able to modify the binding is a really useful mechanism. I'd like to propose a different way to achieve it which I think will be...

I've given this a bit more thought and you don't need to make it public to use it. You can do this instead: ```c# partial internal class HelloServiceClient { public...

Everything so far looks good. Need to update coding style to: * Specify accessibility of members * Use auto properties * One line methods use expression method bodies * Update...

Port Sharing is likely to come around the middle of the year. The WCF Client project recently shipped a preview release of client named pipes which is a pre-requisite for...

@wim-sn, when I release the named pipes support, NetTcp should work hosted in IIS without the port sharing or service activation parts as a side effect of my refactoring everything.

Port sharing isn't available yet, but the latest preview packages have support for hosting NetTcp when not using Kestrel. I just threw together a test program to validate this and...