Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A variable name should not start with $ #575

Open
saig0 opened this issue Nov 28, 2022 · 0 comments
Open

A variable name should not start with $ #575

saig0 opened this issue Nov 28, 2022 · 0 comments

Comments

@saig0
Copy link
Member

saig0 commented Nov 28, 2022

Describe the bug
Currently, a variable name can start with a $ sign. According to the spec, $ is not a valid start of a variable name.

Related to bpmn-io/feel-editor#28.

To Reproduce
Steps to reproduce the behavior:

  1. Evaluate the expression $a with a context {"$a": 1}
  2. Verify that the expression is evaluated successfully

Expected behavior
The $ sign should not be a valid start of a variable name. As a result, an expression should not evaluate successfully (i.e. should not parse successfully) if it contains a variable name that starts with a $ sign.

Environment

  • FEEL engine version: 1.15
  • Affects:
    • Camunda Automation Platform 7: [7.x]
    • Zeebe broker: [0.x]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant