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

Detect cyclic context references #13

Open
JulienCochuyt opened this issue Jul 18, 2019 · 0 comments
Open

Detect cyclic context references #13

JulienCochuyt opened this issue Jul 18, 2019 · 0 comments

Comments

@JulienCochuyt
Copy link
Member

JulienCochuyt commented Jul 18, 2019

There is currently no attempt to detect cyclic context reference, neither in the Rule Editor nor when matching rules.
Detecting in the Rule Editor is for sure the less performance costly.
If the check turns out not to be too heavy on resources, implementing it also when matching rules would be the safest.

Labeling as both "bug" and "enhancement" as:

  • It is a known and deliberate temporary limitation
  • It just turned out to occur in real life (report from Alphonse)
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