-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhance query filters to handle Uuid and enable analysis graph to use filters. #920
Enhance query filters to handle Uuid and enable analysis graph to use filters. #920
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but I'll defer to @jcrossley3
d0c1020
to
7ab0c98
Compare
482d4a4
to
5ad90f4
Compare
5ad90f4
to
9c7b3fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad you figured it out!
9c7b3fc
to
203c987
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you want to fix the graph_read_gaurd
typo while you're in there?
203c987
to
ab2456e
Compare
Enhances query.rs to be able to perform comparisons with Uuid when supplied via inline query params.
Enabling the analysis graph (api/v1/analysis) to filter on sbom_id, node_id and exact name.
where we retain contains style search with:
to filter exact match use:
and similar for dep endpoints eg.
Note - In real usage - the above inline query name=values&name=value should always be escaped.
Most of the 'moving parts' of this feature is due to querying both an ORM and non ORM entity at the same time (sbom_node and analysis graph) ... kept most of the code changes within analysis graph.