Merge pull request #609 from metrico/fix/exception-on-error-log

Fix/exception on error log
metrico · Jan 29, 2025 · c506c3a · c506c3a
2 parents 16f3623 + 9051a17
commit c506c3a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/lib/handlers/otlp_log_push.js b/lib/handlers/otlp_log_push.js
@@ -52,7 +52,7 @@ async function handle (req, res) {
     }
     await Promise.all(promises)
   } catch (error) {
-    await asyncLogError(error)
+    await asyncLogError(error, req.log)
     res.status(500).send({ error: 'Internal Server Error' })
   }
 }

diff --git a/traceql/clickhouse_transpiler/attr_condition.js b/traceql/clickhouse_transpiler/attr_condition.js
@@ -72,7 +72,7 @@ module.exports = class Builder {
       for (const term of self.terms) {
         const sqlTerm = self.getTerm(term)
         self.sqlConditions.push(sqlTerm)
-        if (!term.Child('label_name').value.match(/^(\.|span\.|resource\.|name)/)) {
+        if (!term.Child('label_name').value.match(/^(\.|span\.|resource\.|name|status)/)) {
           continue
         }
         self.where.push(sqlTerm)
@@ -181,6 +181,9 @@ module.exports = class Builder {
         case 'name':
           key = 'name'
           break
+        case 'status':
+          key = 'status'
+          break
         default:
           throw new Error(`unsupported attribute ${key}`)
       }