(LogAgregator): добавлена поддержка иерархии агрегаторов,

Utilities/ULog/log.cc

@@ -20,10 +20,10 @@ static struct option longopts[] =
 	{ "del", required_argument, 0, 'd' },
 	{ "set", required_argument, 0, 's' },
 	{ "off", required_argument, 0, 'o' },
-	{ "on", required_argument, 0, 'n' },
-	{ "list", no_argument, 0, 'm' },
+	{ "on", required_argument, 0, 'e' },
+	{ "list", no_argument, 0, 'l' },
 	{ "rotate", required_argument, 0, 'r' },
-	{ "logname", required_argument, 0, 'l' },
+	{ "logname", required_argument, 0, 'n' },
 	{ "command-only", no_argument, 0, 'b' },
 	{ "timeout", required_argument, 0, 'w' },
 	{ "reconnect-delay", required_argument, 0, 'x' },
@@ -36,7 +36,7 @@ static void print_help()
 	printf("-v, --verbose               - Print all messages to stdout\n");
 	printf("[-i|--iaddr] addr           - LogServer ip or hostname.\n");
 	printf("[-p|--port] port            - LogServer port.\n");
-	printf("[-l|--logname] name         - Send command only for 'logname'.\n");
+	printf("[-n|--logname] name         - Send command only for 'logname'.\n");
 	printf("[-b|--command-only]         - Send command and break. (No read logs).\n");
 	printf("[-w|--timeout] msec         - Timeout for wait data. Default: 0 - endless waiting\n");
 	printf("[-x|--reconnect-delay] msec - Pause for repeat connect to LogServer. Default: 5000 msec.\n");
@@ -48,9 +48,9 @@ static void print_help()
 	printf("[--del | -d] info,warn,crit,...  - Delete log levels.\n");
 	printf("[--set | -s] info,warn,crit,...  - Set log levels.\n");
 	printf("--off, -o                        - Off the write log file (if enabled).\n");
-	printf("--on, -n                         - On the write log file (if before disabled).\n");
+	printf("--on, -e                         - On(enable) the write log file (if before disabled).\n");
 	printf("--rotate, -r                     - rotate log file.\n");
-	printf("--list, -m                       - List of managed logs.\n");
+	printf("--list, -l                       - List of managed logs.\n");
 }
 // --------------------------------------------------------------------------
 int main( int argc, char** argv )
@@ -73,7 +73,7 @@ int main( int argc, char** argv )
 
 	try
 	{
-		while( (opt = getopt_long(argc, argv, "hvma:p:i:d:s:l:onrbx:w:", longopts, &optindex)) != -1 )
+		while( (opt = getopt_long(argc, argv, "hvla:p:i:d:s:n:eorbx:w:", longopts, &optindex)) != -1 )
 		{
 			switch (opt)
 			{
@@ -102,7 +102,7 @@ int main( int argc, char** argv )
 				}
 				break;
 
-				case 'm':
+				case 'l':
 					cmd = LogServerTypes::cmdList;
 					cmdonly = 1;
 					break;
@@ -111,7 +111,7 @@ int main( int argc, char** argv )
 					cmd = LogServerTypes::cmdOffLogFile;
 					break;
 
-				case 'n':
+				case 'e':
 					cmd = LogServerTypes::cmdOnLogFile;
 					break;
 
@@ -123,7 +123,7 @@ int main( int argc, char** argv )
 					addr = string(optarg);
 					break;
 
-				case 'l':
+				case 'n':
 					logname = string(optarg);
 					break;
 
@@ -171,7 +171,7 @@ int main( int argc, char** argv )
 			data = (int)Debug::value(sdata);
 
 			if( verb )
-				cout << "SEND COMMAND: '" << (LogServerTypes::Command)cmd << " data='" << sdata << "'" << endl;
+				cout << "SEND COMMAND: '" << (LogServerTypes::Command)cmd << " data='" << sdata << "'(" << (int)data << ")" << endl;
 		}
 
 		lr.readlogs( addr, port, (LogServerTypes::Command)cmd, data, logname, verb );

Utilities/ULog/logserver.cc

@@ -91,13 +91,12 @@ int main( int argc, char** argv )
 			//            dlog.addLevel( Debug::type(Debug::CRIT | Debug::WARN | Debug::INFO) );
 		}
 
-		auto la = make_shared<LogAgregator>();
-
+		auto la = make_shared<LogAgregator>("la");
+		cerr << "create LogAgregator: " << la->getLogName() << endl;
 
 		auto dlog = make_shared<DebugStream>();
 		dlog->setLogName("dlog");
 		la->add(dlog);
-
 		auto dlog2 = la->create("dlog2");
 
 
@@ -115,11 +114,30 @@ int main( int argc, char** argv )
 
 		}
 
+		auto la2 = make_shared<LogAgregator>("la2");
+		
+		cerr << "create LogAgregator: " << la2->getLogName() << endl;
+		
+		auto dlog3 = la2->create("dlog3");
+		auto dlog4 = la2->create("dlog4");
+		la->add(la2);
+
+
+		if( la->getLog("la2/dlog3") == nullptr )
+		{
+			cerr << "Not found 'la2/dlog3'" << endl;
+			return 1;
+
+		}
+
+
 		LogServer ls(la);
 		ls.setMaxSessionCount(msess);
 
 		dlog->addLevel(Debug::ANY);
 		dlog2->addLevel(Debug::ANY);
+		dlog3->addLevel(Debug::ANY);
+		dlog4->addLevel(Debug::ANY);
 
 		ls.run( addr, port, true );
 
@@ -140,6 +158,14 @@ int main( int argc, char** argv )
 			dlog2->warn() << ": dlog2: WARN message" << endl;
 			dlog2->crit() << ": dlog2: CRIT message" << endl;
 
+			dlog3->info() << ": dlog3: INFO message" << endl;
+			dlog3->warn() << ": dlog3: WARN message" << endl;
+			dlog3->crit() << ": dlog3: CRIT message" << endl;
+
+			dlog4->info() << ": dlog4: INFO message" << endl;
+			dlog4->warn() << ": dlog4: WARN message" << endl;
+			dlog4->crit() << ": dlog4: CRIT message" << endl;
+
 			msleep(delay);
 		}
 

Utilities/codegen/ctl-cpp-common.xsl

@@ -171,48 +171,50 @@
 
 		virtual bool setMsg( UniSetTypes::ObjectId code, bool state = true );
 
-		std::shared_ptr<DebugStream> mylog;
+		inline std::shared_ptr<DebugStream> log(){ return mylog; }
+		inline std::shared_ptr<LogAgregator> logAgregator(){ return loga; }
+
 		void init_dlog( std::shared_ptr<DebugStream> d );
 
         // "синтаксический сахар"..для логов
         #ifndef myinfo 
-        	#define myinfo if( mylog->debugging(Debug::INFO) ) mylog->info() 
+        	#define myinfo if( log()->debugging(Debug::INFO) ) log()->info() 
         #endif
         #ifndef mywarn
-	        #define mywarn if( mylog->debugging(Debug::WARN) ) mylog->warn()
+	        #define mywarn if( log()->debugging(Debug::WARN) ) log()->warn()
         #endif
         #ifndef mycrit
-    	    #define mycrit if( mylog->debugging(Debug::CRIT) ) mylog->crit()
+    	    #define mycrit if( log()->debugging(Debug::CRIT) ) log()->crit()
         #endif
         #ifndef mylog1
-        	#define mylog1 if( mylog->debugging(Debug::LEVEL1) ) mylog->level1()
+        	#define mylog1 if( log()->debugging(Debug::LEVEL1) ) log()->level1()
         #endif
         #ifndef mylog2
-	        #define mylog2 if( mylog->debugging(Debug::LEVEL2) ) mylog->level2()
+	        #define mylog2 if( log()->debugging(Debug::LEVEL2) ) log()->level2()
         #endif
         #ifndef mylog3
-    	    #define mylog3 if( mylog->debugging(Debug::LEVEL3) ) mylog->level3()
+    	    #define mylog3 if( log()->debugging(Debug::LEVEL3) ) log()->level3()
         #endif
         #ifndef mylog4
-        	#define mylog4 if( mylog->debugging(Debug::LEVEL4) ) mylog->level4()
+        	#define mylog4 if( log()->debugging(Debug::LEVEL4) ) log()->level4()
         #endif
         #ifndef mylog5
-	        #define mylog5 if( mylog->debugging(Debug::LEVEL5) ) mylog->level5()
+	        #define mylog5 if( log()->debugging(Debug::LEVEL5) ) log()->level5()
         #endif
         #ifndef mylog6
-    	    #define mylog6 if( mylog->debugging(Debug::LEVEL6) ) mylog->level6()
+    	    #define mylog6 if( log()->debugging(Debug::LEVEL6) ) log()->level6()
         #endif
         #ifndef mylog7
-        	#define mylog7 if( mylog->debugging(Debug::LEVEL7) ) mylog->level7()
+        	#define mylog7 if( log()->debugging(Debug::LEVEL7) ) log()->level7()
         #endif
         #ifndef mylog8
-	        #define mylog8 if( mylog->debugging(Debug::LEVEL8) ) mylog->level8()
+	        #define mylog8 if( log()->debugging(Debug::LEVEL8) ) log()->level8()
         #endif
         #ifndef mylog9
-    	    #define mylog9 if( mylog->debugging(Debug::LEVEL9) ) mylog->level9()
+    	    #define mylog9 if( log()->debugging(Debug::LEVEL9) ) log()->level9()
         #endif
         #ifndef mylogany
-        	#define mylogany mylog->any()
+        	#define mylogany log()->any()
         #endif
         
         // Вспомогательные функции для удобства логирования
@@ -305,7 +307,7 @@
 		bool forceOut; /*!< флаг принудительного обноления "выходов" */
 		
 		std::shared_ptr<LogAgregator> loga;
-		std::shared_ptr<DebugStream> smlog;
+		std::shared_ptr<DebugStream> mylog;
 		std::shared_ptr<LogServer> logserv;
 		std::string logserv_host = {""};
 		int logserv_port = {0};
@@ -391,7 +393,7 @@ void <xsl:value-of select="$CLASSNAME"/>_SK::preSysCommand( const SystemMessage*
 		{
 			// переоткрываем логи
 			mylogany &lt;&lt; myname &lt;&lt; "(preSysCommand): logRotate" &lt;&lt; endl;
-			string fname( mylog->getLogFile() );
+			string fname( log()->getLogFile() );
 			if( !fname.empty() )
 			{
 				mylog->logFile(fname.c_str(),true);
@@ -708,7 +710,7 @@ end_private(false)
 		conf->initLogStream(mylog,s.str());
 	}
 
-	loga = make_shared&lt;LogAgregator&gt;();
+	loga = make_shared&lt;LogAgregator&gt;(myname+"-loga");
 	loga-&gt;add(mylog);
 	loga-&gt;add(ulog());
 
@@ -1099,7 +1101,7 @@ askPause(uniset_conf()->getPIntProp(cnode,"askPause",2000))
         conf->initLogStream(mylog, s.str());
     }
 
-    loga = make_shared&lt;LogAgregator&gt;();
+	loga = make_shared&lt;LogAgregator&gt;(myname+"-loga");
 	loga-&gt;add(mylog);
 	loga-&gt;add(ulog());
 

Utilities/codegen/tests/TestGen.cc

@@ -38,6 +38,16 @@ void TestGen::sensorInfo( const SensorMessage* sm )
 // -----------------------------------------------------------------------------
 void TestGen::timerInfo( const TimerMessage* tm )
 {
+	if( tm->id == 1 )
+	{
+		askTimer(1,0);
+		askTimer(2,3000);
+	}
+	else if( tm->id == 2 )
+	{
+		askTimer(1,2000);
+		askTimer(2,0);
+	}
 }
 // -----------------------------------------------------------------------------
 void TestGen::sigterm( int signo )
@@ -45,3 +55,9 @@ void TestGen::sigterm( int signo )
 	TestGen_SK::sigterm(signo);
 }
 // -----------------------------------------------------------------------------
+void TestGen::sysCommand( const UniSetTypes::SystemMessage* sm )
+{
+	if( sm->command == SystemMessage::StartUp )
+		askTimer(1,2000);
+}
+// -----------------------------------------------------------------------------

Utilities/codegen/tests/TestGen.h

@@ -11,13 +11,13 @@ class TestGen:
 		TestGen( UniSetTypes::ObjectId id, xmlNode* confnode = UniSetTypes::uniset_conf()->getNode("TestGen") );
 		virtual ~TestGen();
 
-
 	protected:
 		TestGen();
 
 		virtual void step() override;
 		virtual void sensorInfo( const UniSetTypes::SensorMessage* sm ) override;
 		virtual void timerInfo( const UniSetTypes::TimerMessage* tm ) override;
+		virtual void sysCommand( const UniSetTypes::SystemMessage* sm ) override;
 		virtual void sigterm( int signo ) override;
 
 	private:

conf/libuniset2.spec

@@ -13,7 +13,7 @@
 
 Name: libuniset2
 Version: 2.1
-Release: alt2
+Release: alt3
 
 Summary: UniSet - library for building distributed industrial control systems
 
@@ -455,6 +455,12 @@ mv -f %buildroot%python_sitelibdir_noarch/* %buildroot%python_sitelibdir/%oname
 # ..
 
 %changelog
+* Sun May 31 2015 Pavel Vainerman <pv@altlinux.ru> 2.1-alt3
+- (LogAgregator):
+  - added support agregator hierarchy
+  - add docs
+  - add new tests
+
 * Sat May 30 2015 Pavel Vainerman <pv@altlinux.ru> 2.1-alt2
 - (optimization): TransportMessage change format
 - (uniset-codegen): minor fixes
@@ -470,7 +476,7 @@ mv -f %buildroot%python_sitelibdir_noarch/* %buildroot%python_sitelibdir/%oname
 - minor fixes
 
 * Sun May 24 2015 Pavel Vainerman <pv@altlinux.ru> 2.0-alt35
-- add supported LogServer for: 
+- added support LogServer for: 
    SharedMemory,RRDServer,MBTCPMaster,MBSlave,UNetExchange,IOControl,
    codegen,DBServer_xxx
 

include/LogAgregator.h

@@ -8,19 +8,120 @@
 #include "DebugStream.h"
 #include "LogServerTypes.h"
 // -------------------------------------------------------------------------
+/*!
+	\page page_LogAgregator Агрегатор логов (LogAgregator)
+
+	- \ref sec_LogA_Comm
+	- \ref sec_LogA_Hierarchy
+	- \ref sec_LogA_Regexp
+
+	\section sec_LogA_Comm Общее описание
+	LogAgregator это класс предназначенный для объединения нескольких DebugStream
+	в один поток. При этом LogAgregator сам является DebugStream и обладает всеми его свойствами.
+
+	LogAgregator позволяет управлять каждым из своих логов в отдельности по имени лога.
+	\code
+	std::shared_ptr<DebugStream> dlog1 = std::make_shared<DebugStream>();
+	std::shared_ptr<DebugStream> dlog2 = std::make_shared<DebugStream>();
+
+	std::shared_ptr<DebugStream> la1 = make_shared<LogAgregator>("la1");
+	la1->add(dlog1,"dlog1");
+	la1->add(dlog1,"dlog2");
+
+	// Работа с логами через агрегатор
+	la1->addLevel("dlog1",Debug::INFO);
+	\endcode
+
+	Помимо этого при помощи агрегатора можно сразу создавать дочерние логи
+	\code
+	auto log10 = la1->create("dlog10");
+	log10->warn() << "WARNING MESSAGE" << endl;
+	\endcode
+
+	\section sec_LogA_Hierarchy Иерархия агрегаторов
+
+	Агрегатор позволяет строить иерархии из агрегаторов.
+	При добавлении дочернего агрегатора, все обращения к подчинённым логам другого агрегатора происходят
+	по имени включающим в себя имя дочернего агрегатора.
+	Пример:
+	\code
+	// Где-то в программе есть DebugStream (логи)
+	std::shared_ptr<DebugStream> dlog1 = std::make_shared<DebugStream>();
+	std::shared_ptr<DebugStream> dlog2 = std::make_shared<DebugStream>();
+	std::shared_ptr<DebugStream> dlog3 = std::make_shared<DebugStream>();
+	std::shared_ptr<DebugStream> dlog4 = std::make_shared<DebugStream>();
+
+	std::shared_ptr<LogAgregator> la1 = make_shared<LogAgregator>("la1");
+	la1->add(dlog1,"dlog1");
+	la1->add(dlog1,"dlog2");
+
+	// Работа с логами через агрегатор
+	la1->addLevel("dlog1",Debug::INFO);
+	..
+	// При этом можно выводить и напрямую в агрегатор
+	la1->info() << "INFO MESSAGE" << endl;
+	...
+	std::shared_ptr<LogAgregator> la2 = make_shared<LogAgregator>("la2");
+	la2->add(dlog1,"dlog3");
+	la2->add(dlog1,"dlog4");
+	// работа напрямую...
+	la2->addLevel("dlog4",Debug::WARN);
+
+	// Добавление второго агрегатора в первый..
+	la1->add(la2);
+	...
+	// теперь для обращения к логам второго агрегатора через первый..
+	// нужно добавлять его имя и разделитель "/" (LogAgregator::sep)
+	la1->addLevel("la2/dlog4",Debug::CRIT);
+	\endcode
+
+	\note Все эти свойства в полной мере используются при управлении логами через LogServer.
+	В обычной "жизни" агрегатор вряд ли особо нужен.
+
+
+	\section sec_LogA_Regexp Управление логами с использованием регулярных выражений
+	Агрегатор позволяет получать список подчинённых ему потоков (DebugStream) по именам удовлетворяющим
+	заданному регулярному выражению. Пример:
+	Предположим что иерархия подчинённых потоков выглядит следующим образом
+	\code
+	log1
+	log2
+	log3
+	loga2/log1
+	loga2/log2
+	loga2/loga3/log1
+	loga2/loga3/log2
+	loga2/loga3/log3
+	...
+	\endode
+	Для управления логами можно получить например список всех подчинённых потоков для loga3
+	\code
+	auto lst = la->getLogList(".*loga3.*");
+	for( auto&& l: lst )
+	{
+		..что-то делать..
+	}
+	\endcode
+	\note Полнота и формат поддерживаемых регулярных выражений зависит от поддержки компилятором стандарта с++11 (класс <regex>).
+*/
+// -------------------------------------------------------------------------
 class LogAgregator:
 	public DebugStream
 {
 	public:
 
-		explicit LogAgregator( Debug::type t = Debug::NONE );
-		explicit LogAgregator( char const* f, Debug::type t = Debug::NONE );
+		const std::string sep={"/"}; /*< раздедитель для имён подчинённых агрегаторов */
+
+		explicit LogAgregator( const std::string& name, Debug::type t );
+		explicit LogAgregator( const std::string& name="" );
 
 		virtual ~LogAgregator();
 
 		virtual void logFile( const std::string& f, bool truncate = false ) override;
 
-		void add( std::shared_ptr<DebugStream> log );
+		void add( std::shared_ptr<LogAgregator> log, const std::string& lname="" );
+		void add( std::shared_ptr<DebugStream> log, const std::string& lname="" );
+
 		std::shared_ptr<DebugStream> create( const std::string& logname );
 
 		// Управление "подчинёнными" логами
@@ -28,13 +129,21 @@ class LogAgregator:
 		void delLevel( const std::string& logname, Debug::type t );
 		void level( const std::string& logname, Debug::type t );
 
-
 		struct LogInfo
 		{
-			LogInfo(): log(0), logfile("") {}
-			LogInfo( std::shared_ptr<DebugStream>& l ): log(l), logfile(l->getLogFile()) {}
+			LogInfo(): log(0), logfile(""),logname("") {}
+			LogInfo( std::shared_ptr<DebugStream>& l, const std::string& lname="" ):
+				log(l), logfile(l->getLogFile()),logname(lname.empty()?l->getLogName():lname) {}
+
 			std::shared_ptr<DebugStream> log;
 			std::string logfile;
+			std::string logname;
+
+			// для сортировки "по алфавиту"
+			inline bool operator < ( const LogInfo& r ) const
+			{
+				return logname < r.logname;
+			}
 		};
 
 		std::shared_ptr<DebugStream> getLog( const std::string& logname );
@@ -45,6 +154,8 @@ class LogAgregator:
 
 	protected:
 		void logOnEvent( const std::string& s );
+		void addLog( std::shared_ptr<DebugStream> l, const std::string& lname );
+		void addLogAgregator( std::shared_ptr<LogAgregator> la, const std::string& lname );
 
 
 	private:

include/LogServerTypes.h

@@ -7,7 +7,7 @@
 // -------------------------------------------------------------------------
 namespace LogServerTypes
 {
-	const unsigned int MAGICNUM = 0x20140904;
+	const unsigned int MAGICNUM = 0x20150531;
 	enum Command
 	{
 		cmdNOP,         /*!< отсутствие команды */

include/LogSession.h

@@ -10,6 +10,7 @@
 #include <mutex>
 #include "Mutex.h"
 #include "DebugStream.h"
+#include "LogAgregator.h"
 #include "PassiveTimer.h"
 // -------------------------------------------------------------------------
 /*! Реализация "сессии" для клиентов LogServer. */
@@ -66,6 +67,7 @@ class LogSession:
 		std::string peername;
 		std::string caddr;
 		std::shared_ptr<DebugStream> log;
+		std::shared_ptr<LogAgregator> alog;
 
 		//        PassiveTimer ptSessionTimeout;
 		FinalSlot slFin;

src/Log/LogAgregator.cc

 #include <memory>
 #include <regex>
+#include <sstream>
 #include "DebugExtBuf.h"
 #include "LogAgregator.h"
 // -------------------------------------------------------------------------
+LogAgregator::LogAgregator( const std::string& name ):
+	LogAgregator(name,Debug::NONE)
+{
+}
+// ------------------------------------------------------------------------
+#if 0
 LogAgregator::LogAgregator( char const* f, Debug::type t ):
 	DebugStream(f, t)
 {
 	delete rdbuf(new teebuf(&internal->fbuf, &internal->sbuf));
 }
+#endif
 // -------------------------------------------------------------------------
-LogAgregator::LogAgregator( Debug::type t ):
+LogAgregator::LogAgregator( const std::string& name, Debug::type t ):
 	DebugStream(t)
 {
+	setLogName(name);
 	delete rdbuf(new teebuf(&internal->nbuf, &internal->sbuf));
 }
 // -------------------------------------------------------------------------
@@ -48,17 +57,73 @@ std::shared_ptr<DebugStream> LogAgregator::create( const std::string& logname )
 	return l;
 }
 // -------------------------------------------------------------------------
-void LogAgregator::add( std::shared_ptr<DebugStream> l )
+void LogAgregator::add( std::shared_ptr<DebugStream> l, const std::string& lname )
+{
+	auto lag = std::dynamic_pointer_cast<LogAgregator>(l);
+	if( lag )
+		addLogAgregator(lag, (lname.empty() ? l->getLogName(): lname) );
+	else
+		addLog(l,(lname.empty() ? l->getLogName(): lname));
+}
+// ------------------------------------------------------------------------
+void LogAgregator::add( std::shared_ptr<LogAgregator> loga, const std::string& lname  )
+{
+	addLogAgregator(loga, (lname.empty() ? loga->getLogName(): lname) );
+}
+// -------------------------------------------------------------------------
+void LogAgregator::addLogAgregator( std::shared_ptr<LogAgregator> la, const std::string& lname )
+{
+	auto i = lmap.find(lname);
+	if( i != lmap.end() )
+		return;
+
+	auto lst = la->getLogList();
+	for( auto&& l: lst )
+	{
+		auto lag = std::dynamic_pointer_cast<LogAgregator>(l.log);
+		if( lag )
+		{
+			std::ostringstream s;
+			s << lname << sep << lag->getLogName();
+			addLogAgregator(lag,s.str()); // рекурсия..
+		}
+		else
+		{
+			std::ostringstream s;
+			s << lname << sep << l.log->getLogName();
+			addLog(l.log,s.str());
+		}
+	}
+}
+// ------------------------------------------------------------------------
+void LogAgregator::addLog( std::shared_ptr<DebugStream> l, const std::string& lname )
 {
-	auto i = lmap.find(l->getLogName());
+	auto i = lmap.find(lname);
 
 	if( i != lmap.end() )
 		return;
 
-	l->signal_stream_event().connect( sigc::mem_fun(this, &LogAgregator::logOnEvent) );
-	lmap[l->getLogName()] = l;
+	bool have = false;
+
+	// пока делаем "не оптимальный поиск" / пробегаем по всему списку, зато не храним дополнительный map /
+	// на то, чтобы не подключаться к одному логу дважды,
+	// иначе будет дублироваие при выводе потом (на экран)
+	for( const auto& i : lmap )
+	{
+		if( i.second.log == l )
+		{
+			have = true;
+			break;
+		}
+	}
+
+	if( !have )
+		l->signal_stream_event().connect( sigc::mem_fun(this, &LogAgregator::logOnEvent) );
+
+	LogInfo li(l,lname);
+	lmap[lname] = li;
 }
-// -------------------------------------------------------------------------
+// ------------------------------------------------------------------------
 void LogAgregator::addLevel( const std::string& logname, Debug::type t )
 {
 	auto i = lmap.find(logname);
@@ -129,7 +194,8 @@ std::list<LogAgregator::LogInfo> LogAgregator::getLogList( const std::string& re
 
 		for( auto && i : lmap )
 		{
-			if( std::regex_match(i.second.log->getLogName(), rule) )
+			//if( std::regex_match(i.second.log->getLogName(), rule) )
+			if( std::regex_match(i.first, rule) )
 				l.push_back(i.second);
 		}
 	}

src/Log/LogReader.cc

@@ -202,6 +202,7 @@ void LogReader::readlogs( const std::string& _addr, ost::tpport_t _port, LogServ
 				{
 					tcp->read(buf, n);
 					buf[n] = '\0';
+
 					log << buf;
 
 					if( msg.cmd == LogServerTypes::cmdList )

src/Log/LogSession.cc

@@ -49,6 +49,10 @@ LogSession::LogSession( ost::TCPSocket& server, std::shared_ptr<DebugStream>& _l
 		log->signal_stream_event().connect( sigc::mem_fun(this, &LogSession::logOnEvent) );
 	else
 		slog.crit() << "LOG NULL!!" << endl;
+
+	auto ag = dynamic_pointer_cast<LogAgregator>(log);
+	if( ag )
+		alog = ag;
 }
 // -------------------------------------------------------------------------
 void LogSession::logOnEvent( const std::string& s )
@@ -109,14 +113,12 @@ void LogSession::run()
 
 				if( !cmdLogName.empty () )
 				{
-					auto lag = dynamic_pointer_cast<LogAgregator>(log);
-
-					if( lag )
+					if( alog ) // если у нас "агрегатор", то работаем с его списком потоков
 					{
 						if( cmdLogName == "ALL" )
-							loglist = lag->getLogList();
+							loglist = alog->getLogList();
 						else
-							loglist = lag->getLogList(cmdLogName);
+							loglist = alog->getLogList(cmdLogName);
 					}
 					else
 					{
@@ -134,21 +136,25 @@ void LogSession::run()
 					ostringstream s;
 					s << "List of managed logs:" << endl;
 					s << "=====================" << endl;
-					auto lag = dynamic_pointer_cast<LogAgregator>(log);
-
-					if( !lag )
+					if( !alog )
 					{
 						s << log->getLogName() << endl;
 					}
 					else
 					{
-						auto lst = lag->getLogList();
+						std::list<LogAgregator::LogInfo> lst;
+						if( cmdLogName.empty() || cmdLogName == "ALL" )
+							lst = alog->getLogList();
+						else
+							lst = alog->getLogList(cmdLogName);
+
+						// красиво отсортируем сперва..
+						lst.sort([](const LogAgregator::LogInfo & a, const LogAgregator::LogInfo & b) { return a.logname < b.logname; });
 
 						for( const auto& i : lst )
-							s << i.log->getLogName() << endl;
+							s << i.logname << endl;
 					}
-
-					s << "=====================" << endl;
+					s << "=====================" << endl << endl;
 
 					if( isPending(Socket::pendingOutput, cmdTimeout) )
 					{
@@ -158,7 +164,7 @@ void LogSession::run()
 					}
 				}
 
-				// обрабатываем команды только если нашли log
+				// обрабатываем команды только если нашли подходящие логи
 				for( auto && li : loglist )
 				{
 					// Обработка команд..

tests/test_logserver.cc

@@ -243,3 +243,25 @@ TEST_CASE("MaxSessions", "[LogServer]" )
 		r2_thr->join();
 }
 // --------------------------------------------------------------------------
+TEST_CASE("Logagregator regexp", "[LogAgregator]" )
+{
+	auto la = make_shared<LogAgregator>();
+	auto log1 = la->create("log1");
+	auto log2 = la->create("log2");
+	auto log3 = la->create("a3/log1");
+	auto log4 = la->create("a3/log2");
+	auto log5 = la->create("a3/log3");
+	auto log6 = la->create("a3/a4/log1");
+	auto log7 = la->create("a3/a4/log2");
+
+	auto lst = la->getLogList(".*/a4/.*");
+	REQUIRE( lst.size() == 2 );
+
+	lst = la->getLogList("a3/.*");
+	REQUIRE( lst.size() == 5 );
+
+	lst = la->getLogList(".*log1.*");
+	REQUIRE( lst.size() == 3 );
+}
+
+// --------------------------------------------------------------------------