Hi,
There are two functions in the TornadoPilotLogging that accept a "wnVO" argument for a fallback case: This should be validated to ensure it doesn't contain any unexpected characters (such as "/" which might cause problems with the lower functions). Could you please add a regexp or similar to check it only contains characters valid in a VO name, ^[A-Za-z0-9-]+(\.[A-Za-z0-9-]+)*$ maybe? (Perhaps it should be normalised to lower-case for consistency everywhere too?).
Regards,
Simon
|
def export_sendMessage(self, message, pilotUUID, wnVO): |
Hi,
There are two functions in the TornadoPilotLogging that accept a "wnVO" argument for a fallback case: This should be validated to ensure it doesn't contain any unexpected characters (such as "/" which might cause problems with the lower functions). Could you please add a regexp or similar to check it only contains characters valid in a VO name,
^[A-Za-z0-9-]+(\.[A-Za-z0-9-]+)*$maybe? (Perhaps it should be normalised to lower-case for consistency everywhere too?).Regards,
Simon
DIRAC/src/DIRAC/WorkloadManagementSystem/Service/TornadoPilotLoggingHandler.py
Line 53 in 6a71b0d