WCF Callbacks Hanging WPF Applications

Skill

WCF Callbacks Hanging WPF Applications

Posted in:

Here's a bug that has driven me nuts over the past few days. I have a WPF application that communicates with a pretty basic WCF service. Whenever a callback is issued in the middle of a request, the WPF application completely hangs. It's obviously a synchronization issue, however I've gone through the forums and articles and set every imaginable attribute on every imaginable object with no successful outcome.

In the end, the answer was unexpectedly simple, however not one I would have ever guessed. I accidentally stumbled across the solution during a round of trying totally crazy things in an attempt to make something work. All I had to do was create my channel on a thread other than the application's main thread. Let me demonstrate with some examples.

public partial class Window1 : Window
{
  IStringReverser _channel;

  public Window1()
  {
    InitializeComponent();

    Callbacks callbacks = new Callbacks(_btnReverse);

    var factory = new DuplexChannelFactory<IStringReverser>(
      callbacks,
      new NetNamedPipeBinding(),
      new EndpointAddress(
         "net.pipe://localhost/PipeReverse"));

    _channel = factory.CreateChannel();
  }

  private void _btnReverse_Click(object sender, RoutedEventArgs e)
  {
    //ReverseString results in the server firing a callback.
    //Making this call will freeze the application.
    _channel.ReverseString("Hello World");
  }
}

The interfaces and most of the code is from a previous tutorial. If the correct attributes are not set on the client's implementation of the callback interface and the server's implementation of the service contract, I would actually expect a deadlock to occur, however the app still hangs regardless of the configuration.

The solution is to simply wrap the CreateChannel call in a new thread.

public partial class Window1 : Window
{
  IStringReverser _channel;

  public Window1()
  {
    InitializeComponent();

    Callbacks callbacks = new Callbacks(_btnReverse);

    var factory = new DuplexChannelFactory<IStringReverser>(
      callbacks,
      new NetNamedPipeBinding(),
      new EndpointAddress(
         "net.pipe://localhost/PipeReverse"));

    ThreadPool.QueueUserWorkItem(new WaitCallback(
      (obj) =>
      {
        _channel = factory.CreateChannel();
      }));
  }

  private void _btnReverse_Click(object sender, RoutedEventArgs e)
  {
    //This call no longer freezes the app
    //and the callback is received correctly.
    _channel.ReverseString("Hello World");
  }
}

The callback now works without hanging the application, but there's one more hurdle. In order to do anything to the user interface, you're going to have to use the dispatcher and invoke a call. If you do this, the application will again hang - invoking onto the UI thread is exactly what got us into this mess in the first place. This can be easily fixed by using the dispatcher's BeginInvoke function.

public void MyCallbackFunction(string callbackValue)
{
  //doing this will hang the application
  _dispatcher.Invoke(new Action(
    () => _someControl.Text = callbackValue;));

  //this will not cause the application to hang
  _dispatcher.BeginInvoke(new Action(
    () => _someControl.Text = callbackValue;));
}

The _dispatcher variable is something you're going to have to get a hold of at some point during the creation of the callback interface. I typically create the interface on the UI thread then simply call Dispatcher.CurrentDispatcher and save that to my variable.

I've read several things stating that all that's actually required is to set the IsOneWay property on the callback's OperationContract attribute. In theory, that's how it should work, since if that is true, WCF will not perform any locking when sending that data. Unfortunately, that doesn't work. You should still keep that property set though, since it does other things you really will want.

The other major piece of information I came across was the UseSynchronizationContext property on the callback implementation's ServiceBehavior attribute. This one sounded really promising. When set to false, the callback will not automatically synchronize with the UI thread. This seems to be the source of the deadlock anyway, so I thought this was definitely it, but again, no luck.

All-in-all it came down to a simple fix that required a lot of headache to find. I'm not totally happy with the outcome, so if anyone else out there experienced this problem and has found a real solution, please let me know.

Anon
04/20/2009 - 14:22

At least in the original example, the callback interface didn't have any service attributes. Adding [CallbackBehavior(ConcurrencyMode=ConcurrencyMode.Multiple, UseSynchronizationContext=false)] solved the apparently same issue for me...

reply

The Reddest
04/24/2009 - 15:55

See, that's why I love this place. I kept applying a ServiceBehavior to the Callback implementation instead of a CallbackBehavior. Your solution works perfectly.

reply

dadkind
11/05/2009 - 16:45

bump.

This solution seems to be the way to go.

This has gotten us past our deadlock issues.

reply

JesusFreak
06/18/2010 - 09:42

This is exactly what I needed, thank you very much!

reply

dadkind
11/05/2009 - 16:46

Sorry, I should have quoted/copied.

This is the solution that worked for us.

04/20/2009 - 14:22

At least in the original example, the callback interface didn't have any service attributes. Adding [CallbackBehavior(ConcurrencyMode=ConcurrencyMode.Multiple, UseSynchronizationContext=false)] solved the apparently same issue for me...

reply

Rich
01/15/2010 - 11:17

thank you so much - just what I needed!

excellent job!!

reply

Romout
01/18/2010 - 11:07

Same for me - I had the problem that a callback method implementation was not able to re-invoke the server. Of course, I've tried it with ____ServiceBehavior____(ConcurrencyMode = ConcurrencyMode.Multiple, UseSynchronizationContext = false) - It took ages until I found Anon's post or better, Reddest's reply to it and realized that I did exactly the same mistake.

reply

Jox
02/11/2010 - 10:40

great small easily added in my solution for a first version..
one thing about the threadpool : does it also handle many callback on duplexcommunicationchannel dcc through different subthread or is it only available for the object dcc ?

reply

DavidB
04/02/2010 - 11:25

Thanks guys. I spent a full day trying to figure out why my client app was hanging. Adding [CallbackBehavior(ConcurrencyMode = ConcurrencyMode.Multiple, UseSynchronizationContext = false)] to the clint class that implements the callback interface fixed my problem.

reply

Naru
04/06/2010 - 07:08

Ah, ah, thank you. I had only UseSynchronizationContext=false.

Added ConcurrencyMode=ConcurrencyMode.Multiple and its working with Invoke() :) Thanks Anon and { }

reply

ViBi
05/20/2010 - 04:57

Thanks a lot. Tum jiyo hazaaron saal (in hindi)

reply

WereWoxel
07/30/2010 - 01:29

Thanks! Excelent solution!

reply

Padraig
08/19/2010 - 12:35

> Adding [CallbackBehavior(ConcurrencyMode=ConcurrencyMode.Multiple, UseSynchronizationContext=false)] solved the apparently same issue for me...

Thank you, thank you, thank you, thank you :)

reply

Add Comment

Put code snippets inside language tags:
[language] [/language]

Examples:
[javascript] [/javascript]
[actionscript] [/actionscript]
[csharp] [/csharp]

See here for supported languages.

Javascript must be enabled to submit anonymous comments - or you can login.

Sponsors