I have the following method in my Hub class:
public class NotificationHub : Hub
{
public void SendAll(string message)
{
if (1 == 0)
Clients.All.broadcastMessage(message); // this should be unreachable
}
}
Then, I (am trying) to call that method from my server-side code like so:
GlobalHost.ConnectionManager.GetHubContext<NotificationHub>()
.Clients.All.broadcastMessage("broadcastMessage was called");
The method is called, and everything works. But, I didn't want broadcastMessage() to be called since it should have been unreachable.
I read this from the documentation:
You don't instantiate the Hub class or call its methods from your own
code on the server; all that is done for you by the SignalR Hubs
pipeline. SignalR creates a new instance of your Hub class each time
it needs to handle a Hub operation such as when a client connects,
disconnects, or makes a method call to the server.
Ref. http://www.asp.net/signalr/overview/signalr-20/hubs-api/hubs-api-guide-server
But it doesn't look like it uses my methods at all. It just looks like it calls its own methods and ignores mine. How can I call my own methods using SignalR?
When you do this:
GlobalHost.ConnectionManager.GetHubContext<NotificationHub>()
.Clients.All.broadcastMessage("broadcastMessage was called");
You're bypassing the method
public void SendAll(string message)
I am not sure why you'd expect the first method to be blocked. If you want your hub logic to work you have to work through the hub methods (example: public void SendAll(string message))
I like the solution presented here: https://stackoverflow.com/a/17897625/693272 if you want to call the hub method from outside.
Related
I am setting my GlobalHost Configuration like this by following this answer to listen when the client is unreachable:
GlobalHost.Configuration.ConnectionTimeout = TimeSpan.FromSeconds(50);
GlobalHost.Configuration.DisconnectTimeout = TimeSpan.FromSeconds(30);
GlobalHost.Configuration.KeepAlive = TimeSpan.FromSeconds(10);
And Overriding the OnDisconnected method in my HUB Class to set the client has been disconnected
public override Task OnDisconnected(bool stopCalled) {
/*My code for saving the information of disconnecting*/
return base.OnDisconnected(stopCalled);
}
I am using Xamarin for android as a client and calling the Stop() method by overriding the OnStop() method of my activity, like this:
protected override void OnStop()
{
//hubConnection.Stop(); previously I was using this but it takes too long to stop the hub connection in this way. so I wrote an explicit method and invoke it .
Task hubconnection = serverHub.Invoke("StopConnection", new object[] { MethodToIdentifyDevice() }).ContinueWith(r =>
{
});
base.OnStop();
}
Secondly, I have written an explicit hubmethod to invoke when to notify my server explicitly that my client has stopped working. That method works at OnStop event.
My actual problem is that what if
All of the stuff above is not able to call OnDisconnected method on activity stop or the application closed.
Is there anything I am missing which is not letting it to happen.
UPDATE:
I have tried changing the Transport level to WebSocket but it is not provided in Xamarin SDK for SignalR as mentioned in the intellisense .
Since WebSocketTransport is not available at Xamarin, I advice to use "Ping" workaround.
Implement Ping() method at server-side.
Call this method from clients periodically (say 1/2 of the timeout interval).
Within this method save ConnectionId : DateTime key/value pair to static ConcurrentDictionary at server-side.
Run background Task at server-side and check DateTime for all dictionary keys.
Remove old-ones and call appropriate code.
I am using Signalr 1.1.4 because im still using .net4 so cant upgrade to signalr 2.
Basically i want to post a message from the server to just the caller to avoid messages being sent to any client that did not start the process off.
My hub class looks like this
public class UpdateHub : Hub
{
/// <summary>
/// Sends the message.
/// </summary>
/// <param name="progressMessage">The progress message.</param>
public void SendMessage(string progressMessage)
{
Clients.Client(Context.ConnectionId).sendMessage(string.Format(progressMessage));
}
}
my javascript looks like this
// get handle to subscriptionUpload hub generated by SignalR
var updateHub = $.connection.UpdateHub;
// establish the connection to the server and start server-side operation
$.connection.hub.start();
updateHub.client.sendMessage = function (message)
{
$("container").empty();
$("container").append(message);
}
Now in my controller action method i would like to do something like this
UpdateHub hub = new UpdateHub();
hub.SendMessage("process has started");
//continue on with long process
hub.SendMessage("process has ended");
Is this even possible?
What we can find in documentation documentation:
You don't instantiate the Hub class or call its methods from your own
code on the server; all that is done for you by the SignalR Hubs
pipeline. SignalR creates a new instance of your Hub class each time
it needs to handle a Hub operation such as when a client connects,
disconnects, or makes a method call to the server.
Because instances of the Hub class are transient, you can't use them
to maintain state from one method call to the next. Each time the
server receives a method call from a client, a new instance of your
Hub class processes the message. To maintain state through multiple
connections and method calls, use some other method such as a
database, or a static variable on the Hub class, or a different class
that does not derive from Hub. If you persist data in memory, using a
method such as a static variable on the Hub class, the data will be
lost when the app domain recycles.
And then:
If you want to send messages to clients from your own code that runs
outside the Hub class, you can't do it by instantiating a Hub class
instance, but you can do it by getting a reference to the SignalR
context object for your Hub class.
You can get the context of your hub: GlobalHost.ConnectionManager.GetHubContext<YourHub>()
and then you can use it to call methods on the client side like this:
context.Clients.All.YourMethod(params);
or
context.Clients.Client(someConnectionID).YourMethod(params);
But in this case you won't be able to use Context.ConnectionId in this methods, because you don't have a direct connection to your hub. In this case you will need to store your connections somewhere (static variables, cache, db etc) and then use it to determine which client should be called.
Hope it will help.
I have inside my hub a method that is used to send to specific clients that a ticket was updated. However I want it to send it to the server inside the controller and to specific users, you can see how I am doing it below:
I am using the following method to send to the specific clients:
public void NotifyUpdatedTicket(Ticket ticket, string note, params string[] aliases)
{
foreach (string a in aliases)
{
foreach (string connectionId in _allConnections.GetConnections(a))
{
Clients.Client(connectionId).notifyUpdatedTicket(ticket, note);
}
}
}
The _allConnections is the same as the class in http://www.asp.net/signalr/overview/guide-to-the-api/mapping-users-to-connections#inmemory
However instead of passing 1 user I passed a list of aliases and then get their connections and send to their connection ids.
However it does not seem to be sending properly it sends it to all clients. Here is the command I am using to call the method in my controller:
GlobalHost.ConnectionManager.GetHubContext<TicketHub>().Clients.All.notifyUpdatedTicket(viewModel.Current, viewModel.NewNote, viewModel.Current.OpenedBy, viewModel.Current.AssignedTo);
When I send the above command with the specific OpenedBy and AssignedTo, it sends it all users when it shouldn't.
Am I missing something from calling it in the controller that sends it only to specific users or something?
Even when I put in the persons alias directly into the hub method it still sends to everyone. I assume this has to do with the controller method.
This
Clients.All.notifyUpdatedTicket
will send a message to all clients. It won't call your method with this signature (I assume that's what you're trying to do):
public void NotifyUpdatedTicket(Ticket ticket, string note, params string[] aliases)
I assume this is a method you added to your hub class.
It's not a good idea to put methods in a hub class if their sole purpose is to be called from the server-side.
Anyway, if you do, you need to use GetHubContext<TicketHub>() from within that method even if it is part of the hub class, because the hub context is set up as part of the hub pipeline when a request to that hub arrives. Without that, the hub is just a regular class and you don't have access to dynamic members like Clients.
So, all in all, it seems like you're confused about some fundamentals. I'd suggest moving NotifyUpdatedTicket to a separate class that also caches the relevant hub context, then call that method when needed (e.g. from a MVC controller method if that is your intent).
According to the documentation I have read, in order to send a message to a client, I just need to call:
var hubContext = GlobalHost.ConnectionManager.GetHubContext<MyHub>();
hubContext.Clients.All.foo(msg);
So... does the Hub class need any methods? If not, then all I have is an empty class:
public class MyHub : Hub
{
}
which just seems like a pointless setup. Am I implementing this incorrectly? because it makes more sense to have methods in a hub class, and then call those methods to send a message to the client.
Also, in the hub itself, I can access Context.connectionId, so that I can get the requestor's connection Id and then stop the message from being fired to that client.... If a Hub shouldn't have methods, then is there a way to access the requestor's connection id?
Yes, you need an empty HUB class declaration, because - It is actually just a proxy between the JS client and the controller so it could be empty since all methods are called via the Clients dynamic variable.
I mean without this , you can work, but you have to write JS for that. This is explained briefly in following link.
For more info refer this link - http://www.asp.net/signalr/overview/signalr-20/hubs-api/hubs-api-guide-javascript-client#genproxy
I'm wondering if it's possible to Cast the result of
var hub = GlobalHost.ConnectionManager.GetHubContext<ChatHub>();
To my actual ChatHub class.
Because GlobalHost.ConnectionManager.GetHubContext<ChatHub>() as ChatHub fails
On my ChatHub class I have a method UpdateTime():
public void SendTimeUpdate(DateTime time, string auth)
{
Clients.All.UpdateTime(time, auth);
}
And I want to call it from my other class. Since I can't cast to ChatHub and invoke the SendUpdate I have to go:
GlobalHost.ConnectionManager.GetHubContext<ChatHub>().Clients.All.UpdateTime(time, auth);
But if I go this road, the method SendTimeUpdate isn't added in the proxy script /signalr/hubs
Is there a solution for this problem? I want to get the typed Hub instance and not call stuff directly on the Clients property of the IHubContext.
No you cannot cast the result of ....GetHubContext<.... to your hub class. Sorry :(.
The GetHubContext approach returns an IHubContext when a Hub is only an IHub.
If you'd like to centralize the logic just make a method that you can call into from your hub and from your external service.
Couldn't your class just create a connection to your hub and call the method that way?