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

[Rules Engine] Consider how to best support correct broadness/specificity #12

Open
jpalawaga opened this issue Feb 15, 2021 · 1 comment

Comments

@jpalawaga
Copy link
Owner

There are a couple ways of going about the rules engine...

  • Store as many thing as possible in the global namespace. Allows for max re-use and max chance that we'll remove as many annoying queryparams as possible.
  • Store as few things as possible in the global namespace. This is probably going to be slightly more performant, but also has the chance to miss other params that might otherwise overlap.

Perhaps one thing that could be done is to allow rules to "crossover." Perhaps we can mark rules as Strict: true/false which will allow us to retain QPs for certain domains.
The inverse of this is also true, we could potentially have a "retain," list so that if one domain uses a QP that is otherwise bad, that we can make sure it is never removed.

This may have implications for the json going forward. So while it might not be super critical to have immediately, it might be worth something to design around just so we won't so quickly need multiple json formats. As the list of things that we cut are small, we're unlikely see much erroneous chopping.

@jpalawaga
Copy link
Owner Author

This is similar to #4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant