IDEV-2299: Validation and error handling improvements

[jbabac]

Change(s):

1. Added RequestValidator to handle validation and error handling

Tests:

1. UT:

2. E2E:

[briluza]

Awesome job on this @jbabac .

Question:

• Should we remove the validations inside the api functions? e.g in iris_investigate

 if not (kwargs or domains):
raise ValueError("Need to define investigation using kwarg filters or domains")
if isinstance(active, bool):
kwargs["active"] = str(active).lower()

*Note (maybe a future enhancement): Should we also do type annotations so that it can help devs or users to see what is expected type in each parameters rather always looking in the openapi spec or using the help()?

[jbabac]

is_body_required

Yes we can.

Awesome job on this @jbabac .

Question:

• Should we remove the validations inside the api functions? e.g in iris_investigate

 if not (kwargs or domains):
raise ValueError("Need to define investigation using kwarg filters or domains")
if isinstance(active, bool):
kwargs["active"] = str(active).lower()

*Note (maybe a future enhancement): Should we also do type annotations so that it can help devs or users to see what is expected type in each parameters rather always looking in the openapi spec or using the help()?

Yes, we can. This has been addressed already.