Bug #15004: Stack overflow error when using invalid JS - Rudder - Issue Tracker

Actions

Copy link

Bug #15004

closed

Stack overflow error when using invalid JS

Added by Nicolas CHARLES over 5 years ago. Updated over 4 years ago.

Status:

Rejected

Priority:

N/A

Assignee:

Nicolas CHARLES

Category:

Web - Config management

Target version:

5.0.19

Pull Request:

Severity:

Major - prevents use of part of Rudder | no simple workaround

UX impact:

User visibility:

Operational - other Techniques | Rudder settings | Plugins

Effort required:

Very Small

Priority:

Name check:

Fix check:

Regression:

Description

I used this JS expression for motd

"This node is managed by Rudder, it's hostname is ${rudder.node.hostname} and hashing it is "+
rudder.hash.md5(${rudder.node.hostname})

this caused Rudder to shut down, because of

rudder.hash.md5(RDR-TST-RandHost596.rudder.local)' for Variable MOTD - please check method call and/or syntax; ReferenceError: "RDR" is not defined in <eval> at line number 2 <- Error with p
arameters expansion for node 'RDR-TST-RandHost150.rudder.local' (91dc8c60-2174-4f58-8869-ce3314ad7c27): When processing directive 'MOTD and pre-login banner': Invalid script 'evaljs:"This no
de is managed by Rudder, it's hostname is RDR-TST-RandHost150.rudder.local and hashing it is "+
rudder.hash.md5(RDR-TST-RandHost150.rudder.local)' for Variable MOTD - please check method call and/or syntax; ReferenceError: "RDR" is not defined in <eval> at line number 2
[2019-05-31 10:01:58] ERROR net.liftweb.actor.ActorLogger - Actor threw an exception
java.lang.StackOverflowError: null
        at net.liftweb.common.Failure.exceptionChain(Box.scala:971)
        at net.liftweb.common.Failure.$anonfun$exceptionChain$2(Box.scala:978)
        at scala.collection.immutable.List.flatMap(List.scala:334)
        at net.liftweb.common.Failure.exceptionChain(Box.scala:978)
        at net.liftweb.common.Failure.$anonfun$exceptionChain$2(Box.scala:978)
        at scala.collection.immutable.List.flatMap(List.scala:334)
        at net.liftweb.common.Failure.exceptionChain(Box.scala:978)
        at net.liftweb.common.Failure.$anonfun$exceptionChain$2(Box.scala:978)
        at scala.collection.immutable.List.flatMap(List.scala:334)
        at net.liftweb.common.Failure.exceptionChain(Box.scala:978)
        at net.liftweb.common.Failure.$anonfun$exceptionChain$2(Box.scala:978)
        at scala.collection.immutable.List.flatMap(List.scala:334)
        at net.liftweb.common.Failure.exceptionChain(Box.scala:978)
        at net.liftweb.common.Failure.$anonfun$exceptionChain$2(Box.scala:978)
        at scala.collection.immutable.List.flatMap(List.scala:334)
        at net.liftweb.common.Failure.exceptionChain(Box.scala:978)
        at net.liftweb.common.Failure.$anonfun$exceptionChain$2(Box.scala:978)

errors should nicely accumulate

Actions

Copy link

Updated by Vincent MEMBRÉ over 5 years ago

Target version changed from 5.0.12 to 5.0.13

Actions

Copy link

Updated by Vincent MEMBRÉ over 5 years ago

Target version changed from 5.0.13 to 5.0.14

Actions

Copy link

Updated by François ARMAND over 5 years ago

Assignee set to François ARMAND
Severity set to Major - prevents use of part of Rudder | no simple workaround
User visibility set to Operational - other Techniques | Rudder settings | Plugins
Priority changed from 0 to 50