Providing inventory_id in different format results in 500ISE about insights-host-inventory

[Glutexo]

Reproduced. Details: The bug is caused by passing the string directly to the SQL query. PostgreSQL complains about querying a UUID field by an invalid value:

sqlalchemy.exc.DataError: (psycopg2.DataError) invalid input syntax for uuid: "bbq"
LINE 3: WHERE hosts.account = 'abc123' AND hosts.id IN ('bbq')

I see two problems here:

• The UUID parameters are not validated.
• The SQL exceptions are not caught. This is actually correct: if an invalid query is built, it is an internal server error.

The validation can be hot-fixed by checking the ID in the view function. But a correct approach would be to define a custom validator for Connexion: https://connexion.readthedocs.io/en/latest/request.html?highlight=data#custom-validators

[Glutexo]

Pinning as this looks like a bug bug to me.

[Glutexo]

This bug is still present. Created a new ticket for this.
https://projects.engineering.redhat.com/browse/RHCLOUD-350

[Glutexo]

This is fixed. To some extent probably by #160.

I checked all the cases when the UUID is passed in the request and all are caught with 400 Bad Request.

[Glutexo]

Created some pull requests to cover all the request UUID validation cases by tests. See #265.