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

Behavior Setting for Matcher #60

Closed
der-workfloh opened this issue Mar 25, 2017 · 0 comments
Closed

Behavior Setting for Matcher #60

der-workfloh opened this issue Mar 25, 2017 · 0 comments
Assignees
Milestone

Comments

@der-workfloh
Copy link
Contributor

der-workfloh commented Mar 25, 2017

Description

The simplify the usage and writing of the whitelisting rules, the user should be able to specify how the given rule is to be interpreted. Not every user is custom to write regular expression, but want to add simple url rules. By allowing to select the way of interpreting the given rules, the normal user will have no trouble of adding simple static rules, and advanced users are able to create complex regex based rules to match their needs.

Tasks

  • Add Matcher Strategies
    • Static (this should be default):
      Rule is used as a literal value and will be added onto the base url for matching.
    • RegEx-All (current behavior):
      Rule is based on regular expression, and will be used for looking up matching anywhere in the current Url.
  • Let admin user decide which strategy to use
@der-workfloh der-workfloh added this to the 2.1 milestone Mar 25, 2017
@der-workfloh der-workfloh changed the title Behavior Setting for RegEx Matcher Behavior Setting for Matcher Mar 25, 2017
@der-workfloh der-workfloh self-assigned this Jun 13, 2017
der-workfloh pushed a commit that referenced this issue Jun 13, 2017
der-workfloh pushed a commit that referenced this issue Jun 13, 2017
der-workfloh pushed a commit that referenced this issue Jun 28, 2017
der-workfloh pushed a commit that referenced this issue Jul 5, 2017
der-workfloh pushed a commit that referenced this issue Jul 5, 2017
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