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

[dev.icinga.com #6570] Disallow side-effect-free r-value expressions in expression lists #1733

Closed
icinga-migration opened this issue Jun 24, 2014 · 7 comments
Labels
area/configuration DSL, parser, compiler, error handling enhancement New feature or request
Milestone

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/6570

Created by gbeutner on 2014-06-24 10:38:01 00:00

Assignee: gbeutner
Status: Resolved (closed on 2015-01-08 08:35:03 00:00)
Target Version: 2.3.0
Last Update: 2015-02-13 06:46:53 00:00 (in Redmine)


Expressions for which we can prove that they're side-effect-free should be disallowed in expression lists:

Example:

{
  "-M"
}

Changesets

2015-01-08 08:34:32 00:00 by gbeutner 0091c70

Disallow side-effect-free r-value expressions in expression lists

fixes #6570

2015-01-08 09:33:45 00:00 by gbeutner 093be8b

Allow side-effect-free expressions in a limited number of cases

refs #6570
@icinga-migration
Copy link
Author

Updated by gbeutner on 2014-06-27 07:58:28 00:00

  • Target Version changed from 2.1.0 to 2.0.1

@icinga-migration
Copy link
Author

Updated by gbeutner on 2014-07-01 11:43:33 00:00

  • Target Version changed from 2.0.1 to 2.1.0

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2014-07-10 08:21:56 00:00

  • Target Version deleted 2.1.0

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2014-11-24 14:26:46 00:00

  • Status changed from New to Rejected

@icinga-migration
Copy link
Author

Updated by gbeutner on 2015-01-08 08:33:18 00:00

  • Status changed from Rejected to Assigned
  • Assigned to set to gbeutner
  • Target Version set to 2.3.0

@icinga-migration
Copy link
Author

Updated by gbeutner on 2015-01-08 08:33:49 00:00

Looks like I've figured out a fairly simple way to do this.

@icinga-migration
Copy link
Author

Updated by gbeutner on 2015-01-08 08:35:03 00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset 0091c70.

@icinga-migration icinga-migration added enhancement New feature or request area/configuration DSL, parser, compiler, error handling labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 2.3.0 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/configuration DSL, parser, compiler, error handling enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant