Fixed some of the compiler warnings.
This commit is contained in:
@ -1,4 +1,3 @@
|
||||
using System.Text.Json;
|
||||
using CommonSocketLibrary.Abstract;
|
||||
using CommonSocketLibrary.Common;
|
||||
using HermesSocketLibrary.Socket.Data;
|
||||
@ -24,23 +23,30 @@ namespace TwitchChatTTS.Hermes.Socket.Handlers
|
||||
_logger = logger;
|
||||
}
|
||||
|
||||
public async Task Execute<Data>(SocketClient<WebSocketMessage> sender, Data data)
|
||||
public Task Execute<Data>(SocketClient<WebSocketMessage> sender, Data data)
|
||||
{
|
||||
if (data is not RequestAckMessage message || message == null)
|
||||
return;
|
||||
return Task.CompletedTask;
|
||||
if (message.Request == null)
|
||||
{
|
||||
_logger.Warning("Received a Hermes request message without a proper request.");
|
||||
return;
|
||||
return Task.CompletedTask;
|
||||
}
|
||||
|
||||
_logger.Debug($"Received a Hermes request message [type: {message.Request.Type}][data: {string.Join(',', message.Request.Data?.Select(entry => entry.Key + '=' + entry.Value) ?? Array.Empty<string>())}]");
|
||||
var json = message.Data?.ToString();
|
||||
if (message.Request.Type == null)
|
||||
{
|
||||
return;
|
||||
_logger.Warning("Request type is null. Unknown what acknowlegement this is for.");
|
||||
return Task.CompletedTask;
|
||||
}
|
||||
if (!string.IsNullOrWhiteSpace(message.Error))
|
||||
{
|
||||
_logger.Warning("An error occured while processing the request on the server: " + message.Error);
|
||||
return Task.CompletedTask;
|
||||
}
|
||||
_manager.Fulfill(message.Request.Type, message.Request.RequestId, json, message.Request.Data);
|
||||
return Task.CompletedTask;
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -13,7 +13,7 @@ namespace TwitchChatTTS.Hermes.Socket.Requests
|
||||
_logger = logger;
|
||||
}
|
||||
|
||||
public void Fulfill(string type, string requestId, string data, IDictionary<string, object>? requestData)
|
||||
public void Fulfill(string type, string requestId, string? data, IDictionary<string, object>? requestData)
|
||||
{
|
||||
if (!_acknowledgements.TryGetValue(type, out var ack))
|
||||
{
|
||||
|
Reference in New Issue
Block a user