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");
}
}
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.
Edit: Check out this solution recommended by an alert reader to
this problem.
[CallbackBehavior(ConcurrencyMode=ConcurrencyMode.Multiple, UseSynchronizationContext=false)]
Comments
Post a Comment