Description of problem:

Tech Debt: Triggered by this disucssion under this PR to migrate RBAC plugin to use the new backend.

Actual results:

The current constructor implementation uses optional argument UrlReader and this method is not great for readability. 

  constructor(
    private readonly discovery: DiscoveryService,
    private readonly pluginIdProvider: PluginIdProvider,
    private readonly logger: LoggerService,
    config: Config,
    urlReader?: UrlReaderService, 
  )

Expected results:

Use 2 objects as the arguments instead: `deps` for mandatory parameters and `optional` for optional ones.

Reproducibility (Always/Intermittent/Only Once):

Always