RBAC: Make it possible to use regex for values

[francoisvandenplas]

• Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
Rbac now handle regex in values
Fixes #300

Is there anything you'd like reviewers to focus on?

How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

• No need to
• Manually (please, describe, if necessary)
• Unit checks
• Integration checks
• Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
• My changes generate no new warnings (e.g. Sonar is happy)
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged

Check out Contributing and Code of Conduct

A picture of a cute animal (not mandatory but encouraged)

[callaertanthony]

Hey ! Any chance to see this PR merged ?

[Haarolean]

Hey ! Any chance to see this PR merged ?

sorry, will take a look this week

[Haarolean]

Thanks for covering this with tests, that's really helpful!
Left a few minor inline comments

[callaertanthony]

hey @Haarolean , is it possible for you to give a look again ? many thanks

[Haarolean]

@callaertanthony hey, good point! We've discussed this: we believe a better approach would be introducing a new boolean param like isRegex (nullable & defaults to false for back compatibility), which will indicate whether we should treat the value as regex or a string literal. This will guarantee backward compatibility AND reduce confusion if multiple fields are used instead. What do you think?

[callaertanthony]

thanks @Haarolean , @francoisvandenplas I created this pr with the change : francoisvandenplas#1

[Haarolean]

thanks @Haarolean , @francoisvandenplas I created this pr with the change : francoisvandenplas#1

could you rebase this branch instead please?

[francoisvandenplas]

Hello,
@Haarolean could you have a look ?
Thanks !

[Haarolean]

just one minor thing and we're good to merge. @francoisvandenplas would you mind updating docs as well?
https://github.com/kafbat/ui-docs

[Haarolean]

@francoisvandenplas @callaertanthony thanks for your contribution! We just need to update docs now