whatap 0.5.2 → 0.5.4

package/lib/conf/config-default.js

@@ -110,8 +110,10 @@ var ConfigDefault = {
     "trace_sampling_enabled": bool("trace_sampling_enabled", true),
     "trace_sampling_tps": bool("trace_sampling_tps", 1000),
 
-    "profile_error_sql_time_max": num("profile_error_sql_time_max", 30000),
     "profile_error_step_enabled": bool('profile_error_step_enabled', true),
+    "profile_error_sql_time_max": num("profile_error_sql_time_max", 30000),
+    "profile_error_sql_fetch_max": num("profile_error_sql_fetch_max", 10000),
+    "profile_error_httpc_time_max": num("profile_error_httpc_time_max", 10000),
     "hook_direct_patch_classes": "",
 
     "log_rotation_enabled": bool('log_rotation_enabled',true),
@@ -266,6 +268,7 @@ var ConfigDefault = {
     "grpc_profile_ignore_method": bool("grpc_profile_ignore_method", true),
 
     "oname_port_postfix_enabled": bool("oname_port_postfix_enabled", false),
+    "ignore_http_lost_connection": bool("ignore_http_lost_connection", false)
 };
 
 ConfigDefault._hook_method_ignore_prefix = ConfigDefault.hook_method_ignore_prefixes.split(',');

package/lib/counter/counter-manager.js

@@ -26,8 +26,7 @@ var CounterPack = require('../pack/counter-pack'),
     DateUtil = require('./../util/dateutil'),
     KubeClient       = require('../kube/kube-client'),
     KubeUtil            = require('../util/kube-util'),
-    Logger              = require('../logger'),
-    Long = require('long');
+    Logger              = require('../logger');
 
 function CounterManager(agent) {
     this.agent = agent;
@@ -90,11 +89,17 @@ CounterManager.prototype.process = function (tasks) {
         var ctx = enumer.nextElement();
         if (ctx.getElapsedTime() > conf.profile_max_time) {
             //TraceContextManager.end(ctx._id);
-            var obj = TraceContextManager.resume(ctx._id);
-            TraceContextManager.addStep("Lost Connection","", ctx);
-            //ctx.status = 5;
-            //ctx.error = StatError.addError(500, "Lost Connection", ctx.service_hash);
-            TraceContextManager.endTrace(ctx);
+            if(conf.getProperty("ignore_http_lost_connection", false) === true){
+                Logger.print("WHATAP-072", "Lost Connection was ignored, " + ctx._id, false);
+                TraceContextManager.end(ctx._id);
+                ctx = null;
+            }else{
+                // var obj = TraceContextManager.resume(ctx._id);
+                TraceContextManager.addStep("Lost Connection","", ctx);
+                //ctx.status = 5;
+                //ctx.error = StatError.addError(500, "Lost Connection", ctx.service_hash);
+                TraceContextManager.endTrace(ctx);
+            }
         }
     }
 

package/lib/observers/http-observer.js

@@ -40,6 +40,8 @@ var TraceContextManager = require('../trace/trace-context-manager'),
     IntKeyLinkedMap = require("../util/intkey-linkedmap");
 const ParamSecurity = require("../util/paramsecurity");
 const {Buffer} = require("buffer");
+const shimmer = require('../core/shimmer');
+const TraceHttpc = require('../trace/trace-httpc');
 
 var _exts=new Set([".css",".js",".png", ".htm", ".html", ".gif", ".jpg", ".css", ".txt", ".ico"]);
 
@@ -207,10 +209,6 @@ HttpObserver.prototype.__createTransactionObserver = function(callback, isHttps,
                     ctx.statusMessage = ctx.error_message;
                 }
 
-                var requestPath = '';
-                if(req.route && req.route.path)
-                    requestPath = req.route.path;
-
                 var shouldEndTransaction = shouldEndCurrentTransaction(not_found_ignore, ctx, res, req.route ? req.route.path : '');
                 if (shouldEndTransaction) {
                     self.__endTransaction(null, ctx, req, res);
@@ -671,110 +669,100 @@ HttpObserver.prototype.inject = function( mod, moduleName ) {
 
     if( conf.getProperty('httpc_enabled', true) === false ) { return; }
 
-    aop.both(mod, 'request',
-        function (obj, args, lctx) {
-            var ctx = lctx.context;
-            if (ctx == null || (args[0].host == null && args[0].hostname == null) ) { return; } // for axios
-
-            var isHttpsRepeat = false;
-            if(moduleName === 'https') {
-                args[0].__isHttps = true;
+    shimmer.wrap(mod, 'request', function (original) {
+        return function wrappedRequest(options, callback) {
+            var ctx = TraceContextManager.getCurrentContext();
+            if(!ctx || (!options.host && !options.hostname)){
+                return original.apply(this, arguments);
             }
 
-            if(moduleName === 'http' && args[0].__isHttps) {
-                isHttpsRepeat = true;
+            var isHttpRepeat = false;
+            if(moduleName === 'https'){
+                options.__isHttps = true;
+            }
+            if(moduleName === 'http' && options.__isHttps){
+                isHttpRepeat = true;
             }
 
-            if(!isHttpsRepeat) {
-                try {
-                    if(args[0].method === 'OPTION') {
-                        return;
-                    }
-
-                    var dataConsumption = false;
-
-                    if(args.length > 0) {
-                        var info = args[0];
-
-                        interTxTraceAutoOn(ctx);
-
-                        if(conf.mtrace_enabled){
-                            if(info.headers){
-                                info.headers['x-wtap-po']=transferPOID(ctx);
-                            }else{
-                                info.headers={
-                                    'x-wtap-po': transferPOID(ctx)
-                                };
-                            }
-                            if(conf.stat_mtrace_enabled){
-                                info.headers[conf._trace_mtrace_spec_key1]=transferSPEC_URL(ctx);
-                            }
-                            if(conf.mtid_mtrace_enabled && ctx.mtid.isZero()===false){
-                                info.headers[conf._trace_mtrace_caller_key]=transferMTID_CALLERTX(ctx);
-                            }
+            var step = new HttpStepX();
+            step.start_time = ctx.getElapsedTime();
 
+            if(!isHttpRepeat){
+                if(options.method === 'OPTION'){
+                    return original.apply(this, arguments);
+                }
+                try{
+                    interTxTraceAutoOn(ctx);
+                    if(conf.getProperty('mtrace_enabled', false)){
+                        if(options.headers){
+                            options.headers['x-wtap-po'] = transferPOID(ctx);
+                        }else{
+                            options.headers = {'x-wtap-po': transferPOID(ctx)};
+                        }
+                        if(conf.stat_mtrace_enabled){
+                            options.headers[conf._trace_mtrace_spec_key1]=transferSPEC_URL(ctx);
+                        }
+                        if(conf.mtid_mtrace_enabled && ctx.mtid.isZero()===false){
+                            options.headers[conf._trace_mtrace_caller_key]=transferMTID_CALLERTX(ctx);
                         }
-                        ctx.httpc_url = info.path || '/';
-                        ctx.httpc_host = info.host || info.hostname || '';
-                        ctx.httpc_port = info.port || -1;
                     }
+
+                    ctx.httpc_url = options.path || '/';
+                    ctx.httpc_host = options.host || options.hostname || '';
+                    ctx.httpc_port = options.port || -1;
+
                     ctx.footprint('Http Call Start');
 
                     if (ctx.httpc_port < 0) { ctx.httpc_port = 80 };
-
-                } catch(e) {
-                    return Logger.printError('WHATAP-852', 'Http Repeat ', e, true);
+                }catch (e) {
+                    Logger.printError('WHATAP-852', 'Http Repeat ', e, true);
+                    return original.apply(this, arguments);
                 }
+            }else{
+                return original.apply(this, arguments);
+            }
 
-                aop.functionHook(args, -1,  function(obj, args) {
-                    if(TraceContextManager.resume(ctx._id) == null) { return; }
+            var wrappedCallback;
+            if (typeof callback === 'function') {
+                wrappedCallback = function(response) {
+                    if (TraceContextManager.resume(ctx._id) === null) {
+                        return callback.apply(this, arguments);
+                    }
 
-                    var res = args[0];
-                    aop.before(res, ['on', 'addListener'], function(obj, args) {
-                        if(!dataConsumption && args[0] === 'data') {
-                            dataConsumption = true;
+                    response.on('end', function() {
+                        step.elapsed = ctx.getElapsedTime() - step.start_time;
+                        step.url = HashUtil.hashFromString(ctx.httpc_url);
+                        DataTextAgent.HTTPC_URL.add(step.url, ctx.httpc_url);
+                        step.host = HashUtil.hashFromString(ctx.httpc_host);
+                        DataTextAgent.HTTPC_HOST.add(step.host, ctx.httpc_host);
+                        step.port = ctx.httpc_port;
+
+                        if (response.statusCode >= 400 && transaction_status_error_enable) {
+                            step.error = StatError.addError(response.statusCode, response.statusMessage,
+                                ctx.service_hash, TextTypes.HTTPC_URL, step.url);
+                            if (ctx.error.isZero()) {
+                                ctx.error = step.error;
+                                ctx.statusCode = response.statusCode;
+                                ctx.statusMessage = response.statusMessage;
+                            }
                         }
-                    });
-                    aop.before(res, 'pipe', function() {
-                        dataConsumption = true;
-                    });
-                    aop.before(res, 'resume', function() {
-                        dataConsumption = true;
+
+                        ctx.active_httpc_hash = step.url;
+                        ctx.profile.push(step);
+                        endHttpc(ctx, step);
                     });
 
-                    if (dataConsumption) { return; }
-                    // endHttpc(ctx, step);
-                });
-            }
-        },
-        function (obj, args, ret, lctx) {
-            if(moduleName === 'http' && args[0].__isHttps) {
-                return;
+                    return callback.apply(this, arguments);
+                };
             }
 
-            var ctx = lctx.context;
-            if (ctx == null || ( args[0].host == null && args[0].hostname == null)) { return; }
-            var is_ignore_error = false;
-
-            ret.on('response', function(response){
-                var statusCode = response.statusCode;
-                if(transaction_status_error_enable && statusCode >= 400){
-                    if (step.error.isZero()) {
-                        step.error = StatError.addError(statusCode, response.statusMessage , ctx.service_hash,
-                            TextTypes.HTTPC_URL, step.url);
-                        if (ctx.error.isZero()) {
-                            ctx.error = step.error;
-                            ctx.statusCode = statusCode;
-                            ctx.statusMessage = response.statusMessage;
-                        }
-                    }
-                }
-            })
-            ret.on('error', function(err) {
-                if (TraceContextManager.resume(ctx._id) == null) { return; }
+            var req = original.apply(this, [options, wrappedCallback]);
+
+            req.on('error', function (err) {
+                if (TraceContextManager.resume(ctx._id) === null) { return; }
                 ctx.is_httpc_error = true;
 
-                if (transaction_status_error_enable && step.error.isZero() && !is_ignore_error) {
+                if (transaction_status_error_enable && step.error.isZero()) {
                     step.error = StatError.addError(err.code, err.message, ctx.service_hash,
                         TextTypes.HTTPC_URL, step.url);
                     if (ctx.error.isZero()) {
@@ -787,63 +775,45 @@ HttpObserver.prototype.inject = function( mod, moduleName ) {
                 endHttpc(ctx, step);
             });
 
-            var step = new HttpStepX();
-            step.start_time = ctx.getElapsedTime();
-            step.url = HashUtil.hashFromString(ctx.httpc_url);
-            DataTextAgent.HTTPC_URL.add(step.url, ctx.httpc_url);
-            step.host = HashUtil.hashFromString(ctx.httpc_host);
-            DataTextAgent.HTTPC_HOST.add(step.host, ctx.httpc_host);
-            step.port = ctx.httpc_port;
-
-            aop.before(ret, 'write', function (obj, args) {
-                if(conf.profile_httpc_parameter_enabled == true) {
-                    if(args.length == 1) {
-                        var step = new MessageStep();
-                        step.hash = HashUtil.hashFromString("HTTPC-REQUEST-BODY");
-                        step.start_time = ctx.getElapsedTime();
-                        step.desc = args[0];
-                        DataTextAgent.MESSAGE.add(step.hash, "HTTPC-REQUEST-BODY");
-                        ctx.profile.push(step);
+            shimmer.wrap(req, 'write', function (original) {
+                return function wrappedWrite() {
+                    try {
+                        if (conf.getProperty('profile_httpc_parameter_enabled', true)) {
+                            if (arguments && arguments[0]) {
+                                var bodyData;
+
+                                if (typeof arguments[0] === 'object' && !(arguments[0] instanceof Buffer)) {
+                                    bodyData = JSON.stringify(arguments[0]);
+                                }
+                                if (arguments[0] instanceof Buffer) {
+                                    bodyData = arguments[0].toString('utf8');
+                                }
+                                if(bodyData){
+                                    var step = new MessageStep();
+                                    step.hash = HashUtil.hashFromString("HTTPC-REQUEST-BODY");
+                                    step.start_time = ctx.getElapsedTime();
+                                    step.desc = bodyData;
+                                    DataTextAgent.MESSAGE.add(step.hash, "HTTPC-REQUEST-BODY");
+                                    ctx.profile.push(step);
+                                }
+                            }
+                        }
+                    } catch (e) {
+                        Logger.printError('WHATAP-615', 'HTTP Write hook failed', e, false);
                     }
+                    return original.apply(this, arguments);
                 }
             });
 
-            aop.before(ret, 'end', function (obj, args) {
-                ctx.active_httpc_hash = step.url;
-                ctx.profile.push(step);
-            });
-
-            ret.on('socket', function(socket) {
-                // socket reuse chk
-                if(conf.setSocketChk(socket) == false) {
-                    aop.before(socket, 'end', function (obj, args) {
-                        if(ctx == null) { return; }
-                        TraceContextManager.resume(ctx._id);
-                        endHttpc(ctx, step);
-                    });
-
-                    socket.on('timeout', function () {
-                        if (TraceContextManager.resume(ctx._id) == null) { return; }
-                        is_ignore_error = true;
-                        // if (step.error.isZero()) {
-                        //     var msgObj = { 'class': 'Timeout', 'msg': 'Timeout' };
-                        //     step.error = StatError.addError('Timeout','Timeout', ctx.service_hash);
-                        //     if (transaction_status_error_enable && ctx.error.isZero()) {
-                        //         ctx.error = step.error;
-                        //         ctx.statusCode = 'Timeout';
-                        //         ctx.statusMessage = 'Timeout';
-                        //     }
-                        // }
-                        // endHttpc(ctx, step);
-                    });
-                }
-            });
-        });
+            return req;
+        }
+    });
 
     function endHttpc(ctx, step) {
         if(ctx == null || step == null) { return; }
 
         step.elapsed = ctx.getElapsedTime() - step.start_time;
+        TraceHttpc.isSlowHttpc(ctx, step);
 
         ctx.httpc_count++;
         ctx.httpc_time += step.elapsed;

package/lib/observers/maria-observer.js

@@ -23,7 +23,8 @@ var TraceContextManager = require('../trace/trace-context-manager'),
     conf                = require('../conf/configure'),
     DateUtil            = require('../util/dateutil'),
     Buffer              = require('buffer').Buffer,
-    shimmer             = require('../core/shimmer');
+    shimmer             = require('../core/shimmer'),
+    TraceSQL            = require('../trace/trace-sql');
 
 var MariaObserver = function (agent) {
     this.agent = agent;
@@ -46,6 +47,7 @@ MariaObserver.prototype.inject = function (mod, moduleName) {
 
 var _finishQuery = function (ctx, sql_step) {
     sql_step.elapsed = ctx.getElapsedTime() - sql_step.start_time;
+    TraceSQL.isSlowSQL(ctx);
 
     MeterSql.add(sql_step.hash, sql_step.elapsed, false);
     StatSql.addSqlTime(ctx, ctx.service_hash, sql_step.dbc, sql_step.hash, sql_step.elapsed, false, 0);
@@ -53,6 +55,7 @@ var _finishQuery = function (ctx, sql_step) {
 
 var _handleError = function (ctx, sql_step, err) {
     sql_step.elapsed = ctx.getElapsedTime() - sql_step.start_time;
+    TraceSQL.isSlowSQL(ctx);
 
     MeterSql.add(sql_step.hash, sql_step.elapsed, false);
     StatSql.addSqlTime(ctx, ctx.service_hash, sql_step.dbc, sql_step.hash, sql_step.elapsed, true, 0);
@@ -192,9 +195,23 @@ function createQueryHook(agent) {
             }
 
             const result = original.apply(this, args);
-            // return result;
+
             return result.then(res => {
-                _finishQuery(ctx, sql_step);
+                if (Array.isArray(res) && psql && psql.type === "S") {
+                    var result_step = new ResultSetStep();
+                    result_step.start_time = ctx.getElapsedTime();
+                    result_step.elapsed = 0;
+                    result_step.fetch = res.length;
+                    result_step.sqlhash = psql.sql;
+                    result_step.dbc = dbc_hash;
+                    ctx.profile.push(result_step);
+                    MeterSql.addFetch(result_step.dbc, result_step.fetch, 0);
+                    StatSql.addFetch(result_step.dbc, result_step.sqlhash, result_step.fetch, 0);
+
+                    TraceSQL.isTooManyRecords(sql_step, result_step.fetch, ctx);
+                }
+                var isSelectQuery = psql && psql.type === "S"? true : false;
+                _finishQuery(ctx, sql_step, isSelectQuery, res);
                 return res;
             }).catch(err => {
                 _handleError(ctx, sql_step, err)

package/lib/observers/mssql-observer.js

@@ -19,7 +19,8 @@ var TraceContextManager = require('../trace/trace-context-manager'),
     TextTypes           = require('../lang/text-types'),
     Long                = require('long'),
     conf                = require('../conf/configure'),
-    Logger              = require('../logger');
+    Logger              = require('../logger'),
+    TraceSQL = require('../trace/trace-sql')
 const DateUtil = require('../util/dateutil');
 const MessageStep = require("../step/message-step");
 // const ResourceProfile = require("../util/resourceprofile");
@@ -117,27 +118,25 @@ MssqlObserver.prototype.inject = function (mod, modName) {
                     }
 
                     sql_step.elapsed = ctx.getElapsedTime() - sql_step.start_time;
+                    TraceSQL.isSlowSQL(ctx);
 
                     MeterSql.add(dbc_hash, sql_step.elapsed, args[0] != null);
                     StatSql.addSqlTime(ctx.service_hash, sql_step.dbc,
                         sql_step.hash, sql_step.elapsed, args[0] != null, 0);
 
-                    // if(conf.getProperty('profile_sql_resource_enabled', false)){
-                    //     sql_step.start_cpu = ResourceProfile.getCPUTime();
-                    //     sql_step.start_mem = ResourceProfile.getUsedHeapSize();
-                    // }
-
                     if(psql && psql.type == 'S') {
                         var result_step = new ResultSetStep();
                         result_step.dbc = sql_step.dbc;
                         result_step.sqlhash =  sql_step.hash;
                         try {
-                            result_step.fetch = args[1].recordset.length;
+                            result_step.fetch = args[1][0].length;
                         } catch(e) {
                         }
                         ctx.profile.push(result_step);
                         MeterSql.addFetch(dbc_hash, result_step.fetch, 0);
                         StatSql.addFetch(dbc_hash, result_step.sqlhash, result_step.fetch, 0);
+
+                        TraceSQL.isTooManyRecords(sql_step, result_step.fetch, ctx);
                     } else if(psql && psql.type == 'U') {
 
                     }

package/lib/observers/mysql-observer.js

@@ -22,8 +22,8 @@ var TraceContextManager = require('../trace/trace-context-manager'),
     Logger = require('../logger'),
     conf = require('../conf/configure'),
     DateUtil = require('../util/dateutil'),
-    Buffer = require('buffer').Buffer;
-// const ResourceProfile = require('../util/resourceprofile');
+    Buffer = require('buffer').Buffer,
+    TraceSQL = require('../trace/trace-sql');
 
 var MysqlObserver = function (agent) {
     this.agent = agent;
@@ -162,6 +162,8 @@ var queryHook = function (dbc, agent) {
             sql_step.elapsed = ctx.getElapsedTime() - sql_step.start_time;
             ctx.sql_time += sql_step.elapsed;
 
+            TraceSQL.isSlowSQL(ctx);
+
             // if(conf.getProperty('profile_sql_resource_enabled', false)){
             //     sql_step.start_cpu = ResourceProfile.getCPUTime();
             //     sql_step.start_mem = ResourceProfile.getUsedHeapSize();
@@ -183,6 +185,8 @@ var queryHook = function (dbc, agent) {
                 ctx.profile.push(result_step);
                 MeterSql.addFetch(result_step.dbc, result_step.fetch, 0);
                 StatSql.addFetch(result_step.dbc, result_step.sqlhash, result_step.fetch, 0);
+
+                TraceSQL.isTooManyRecords(sql_step, result_step.fetch, ctx);
             }
 
             if (args[1] != null && psql != null && psql.type === 'U') {

package/lib/observers/pgsql-observer.js

@@ -20,8 +20,10 @@ var TraceContextManager = require('../trace/trace-context-manager'),
     ParamSecurity       = require('../util/paramsecurity'),
     Logger              = require('../logger'),
     Buffer              = require('buffer').Buffer,
-    DateUtil            = require('../util/dateutil');
+    DateUtil            = require('../util/dateutil'),
+    TraceSQL                 = require('../trace/trace-sql');
 const shimmer = require('../core/shimmer');
+const ResultSetStep = require("../step/resultset-step");
 
 var PgSqlObserver = function (agent) {
     this.agent = agent;
@@ -170,11 +172,23 @@ PgSqlObserver.prototype.inject = function (mod, moduleName) {
             }
 
             try {
-                // SQL 실행
                 const result = original.apply(this, arguments);
 
                 if (result && result.then) {
                     return result.then(res => {
+                        if(res.command && res.command === 'SELECT'){
+                            var result_step = new ResultSetStep();
+                            result_step.start_time = ctx.getElapsedTime();
+                            result_step.elapsed = 0;
+                            result_step.fetch = res.rowCount;
+                            result_step.sqlhash = psql.sql;
+                            result_step.dbc = dbc_hash;
+                            ctx.profile.push(result_step);
+                            MeterSql.addFetch(result_step.dbc, result_step.fetch, 0);
+                            StatSql.addFetch(result_step.dbc, result_step.sqlhash, result_step.fetch, 0);
+
+                            TraceSQL.isTooManyRecords(sql_step, result_step.fetch, ctx);
+                        }
                         self._finishQuery(ctx, sql_step);
                         return res;
                     }).catch(err => {
@@ -211,25 +225,11 @@ PgSqlObserver.prototype.inject = function (mod, moduleName) {
             }
         };
     });
-
-    // if (mod.Pool && mod.Pool.prototype.query) {
-    //     shimmer.wrap(mod.Pool.prototype, 'query', function (original) {
-    //         return function () {
-    //             const traceContext = TraceContextManager.getCurrentContext();
-    //             if (!traceContext) {
-    //                 return original.apply(this, arguments);
-    //             }
-    //
-    //             // 동일한 방식으로 처리
-    //             return self.agent.pgClient.query.apply(this, arguments);
-    //         };
-    //     });
-    // }
 };
 
-// 쿼리 완료 처리
 PgSqlObserver.prototype._finishQuery = function (ctx, sql_step) {
     sql_step.elapsed = ctx.getElapsedTime() - sql_step.start_time;
+    TraceSQL.isSlowSQL(ctx);
 
     MeterSql.add(sql_step.hash, sql_step.elapsed, false);
     StatSql.addSqlTime(ctx, ctx.service_hash, sql_step.dbc, sql_step.hash, sql_step.elapsed, false, 0);
@@ -237,8 +237,8 @@ PgSqlObserver.prototype._finishQuery = function (ctx, sql_step) {
 
 PgSqlObserver.prototype._handleError = function (ctx, sql_step, err) {
     sql_step.elapsed = ctx.getElapsedTime() - sql_step.start_time;
+    TraceSQL.isSlowSQL(ctx);
 
-    // Logger.printError('WHATAP-192', 'PGSQL Query error.', err);
     MeterSql.add(sql_step.hash, sql_step.elapsed, false);
     StatSql.addSqlTime(ctx, ctx.service_hash, sql_step.dbc, sql_step.hash, sql_step.elapsed, true, 0);
 };

package/lib/trace/trace-httpc.js

@@ -0,0 +1,26 @@
+const   HttpStepX = require('../step/http-stepx'),
+        conf = require('../conf/configure'),
+        StatError= require('../stat/stat-error'),
+        TextTypes= require("../lang/text-types");
+
+var TraceHttpc = function () {}
+
+TraceHttpc.prototype.hasHttpStepX = function (buffer) {
+    if (!Array.isArray(buffer)) {
+        return [];
+    }
+
+    return buffer.filter(item => item instanceof HttpStepX);
+}
+
+TraceHttpc.prototype.isSlowHttpc = function (ctx, httpc_step) {
+    if(!ctx.profile.buffer)
+        return;
+    var profile_error_httpc_time_max = conf.getProperty('profile_error_httpc_time_max', 10000);
+
+    if(httpc_step.elapsed && (httpc_step.elapsed > profile_error_httpc_time_max)){
+        httpc_step.error = StatError.addError('SLOW_HTTPC', 'SLOW HTTPC', ctx.service_hash, TextTypes.HTTPC_URL, null);
+    }
+}
+
+module.exports = new TraceHttpc();

package/lib/trace/trace-sql.js

@@ -0,0 +1,45 @@
+const   SqlStepX        = require('../step/sql-stepx'),
+    conf            = require('../conf/configure'),
+    StatError       = require('../stat/stat-error'),
+    TextTypes       = require("../lang/text-types");
+
+var TraceSql = function () {}
+
+TraceSql.prototype.hasSqlStepX = function (buffer) {
+    if (!Array.isArray(buffer)) {
+        return [];
+    }
+
+    return buffer.filter(item => item instanceof SqlStepX);
+}
+
+TraceSql.prototype.isSlowSQL = function (ctx) {
+    if(!ctx.profile.buffer)
+        return;
+
+    var profile_error_sql_time_max = conf.getProperty('profile_error_sql_time_max', 30000);
+    var sqlStepX = this.hasSqlStepX(ctx.profile.buffer);
+    var slow_sql_step = sqlStepX.find(step => step.elapsed > profile_error_sql_time_max);
+    if(slow_sql_step){
+        slow_sql_step.error = StatError.addError('SLOW_SQL', 'SLOW SQL', ctx.service_hash, TextTypes.SQL, null);
+        if(ctx.error.isZero()){
+            ctx.error = slow_sql_step.error;
+        }
+    }
+    return;
+}
+
+TraceSql.prototype.isTooManyRecords = function (sql_step, rs_count, ctx) {
+    if(!ctx.profile.buffer)
+        return;
+
+    var profile_error_sql_fetch_max = conf.getProperty('profile_error_sql_fetch_max', 10000);
+    if(rs_count && rs_count > profile_error_sql_fetch_max){
+        sql_step.error = StatError.addError('TOO_MANY_RECORDS', 'Too many records', ctx.service_hash, TextTypes.SQL, null)
+        if(ctx.error.isZero()){
+            ctx.error = sql_step.error;
+        }
+    }
+}
+
+module.exports = new TraceSql();

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "whatap",
   "homepage": "http://www.whatap.io",
-  "version": "0.5.2",
-  "releaseDate": "20241017",
+  "version": "0.5.4",
+  "releaseDate": "20241113",
   "description": "Monitoring and Profiling Service",
   "main": "index.js",
   "scripts": {},