From b3dea6fb9102e1ea7bebbd2a950736ad138e09ff Mon Sep 17 00:00:00 2001 From: JustArchi Date: Thu, 2 Mar 2017 21:51:36 +0100 Subject: [PATCH] Closes #484 --- ArchiSteamFarm/WCF.cs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/ArchiSteamFarm/WCF.cs b/ArchiSteamFarm/WCF.cs index 42483ba9e..9227fafdc 100644 --- a/ArchiSteamFarm/WCF.cs +++ b/ArchiSteamFarm/WCF.cs @@ -203,7 +203,8 @@ namespace ArchiSteamFarm { case GlobalConfig.EWCFBinding.NetTcp: result = new NetTcpBinding { // This is a balance between default of 8192 which is not enough, and int.MaxValue which is prone to DoS attacks - ReaderQuotas = { MaxStringContentLength = ushort.MaxValue }, + // We assume maximum of 1024 characters per each bot included in the response, and maximum of 255 bots overall + ReaderQuotas = { MaxStringContentLength = byte.MaxValue * 1024 }, // We use SecurityMode.None for Mono compatibility // Yes, also on Windows, for Mono<->Windows communication @@ -214,14 +215,16 @@ namespace ArchiSteamFarm { case GlobalConfig.EWCFBinding.BasicHttp: result = new BasicHttpBinding { // This is a balance between default of 8192 which is not enough, and int.MaxValue which is prone to DoS attacks - ReaderQuotas = { MaxStringContentLength = ushort.MaxValue } + // We assume maximum of 1024 characters per each bot included in the response, and maximum of 255 bots overall + ReaderQuotas = { MaxStringContentLength = byte.MaxValue * 1024 } }; break; case GlobalConfig.EWCFBinding.WSHttp: result = new WSHttpBinding { // This is a balance between default of 8192 which is not enough, and int.MaxValue which is prone to DoS attacks - ReaderQuotas = { MaxStringContentLength = ushort.MaxValue }, + // We assume maximum of 1024 characters per each bot included in the response, and maximum of 255 bots overall + ReaderQuotas = { MaxStringContentLength = byte.MaxValue * 1024 }, // We use SecurityMode.None for Mono compatibility // Yes, also on Windows, for Mono<->Windows communication