Follow-up from "Refactor x_ray/scan with CloudConnector::AvailableServices"
The following discussion from !152407 (merged) should be addressed:
-
@mikolaj_wawrzyniak started a discussion: (+1 comment) issue: This is not directly related to X-Ray, However I wonder if namespaces being an optional parameter does not creates vulnerability, where namespace missing might result from bug, which would allow for access to features where it should be denied.
suggestion: Have you considered implementing more strict check in this base class, and relaxing it only in applicable child class (
ee/lib/cloud_connector/self_managed/available_service_data.rb
), wdyt?