Rust: Add Postgres crate Models

[GeekMasher]

This PR is to add support for Postgres SQL Injection sinks

Requires this PR merging first:

• Rust: Add Models As Data Sinks for SQL Injection #18535

Pull Request checklist

All query authors

• A change note is added if necessary. See the documentation in this repository.
• All new queries have appropriate .qhelp. See the documentation in this repository.
• QL tests are added if necessary. See Testing custom queries in the GitHub documentation.
• New and changed queries have correct query metadata. See the documentation in this repository.

Internal query authors only

• Autofixes generated based on these changes are valid, only needed if this PR makes significant changes to .ql, .qll, or .qhelp files. See the documentation (internal access required).
• Changes are validated at scale (internal access required).
• Adding a new query? Consider also adding the query to autofix.

[geoffw0]

LGTM, great to have models for this SQL implementation.

[geoffw0]

The test failures:

+| main.rs:11:25:11:37 | //... | Missing result: sql-sink |
...

I think these will go away if you merge main into the branch, now that the dependency #18535 is in main.

-| file://:0:0:0:0 | [summary param] 0 in lang:alloc::::crate::fmt::format | file://:0:0:0:0 | [summary] to write: ReturnValue in lang:alloc::::crate::fmt::format | MaD:14 |
...
+| file://:0:0:0:0 | [summary param] 0 in lang:alloc::::crate::fmt::format | file://:0:0:0:0 | [summary] to write: ReturnValue in lang:alloc::::crate::fmt::format | MaD:24 |
...

We're aware of this issue, it's a problem with the amount of data we're including in test outputs. I think @hvitved has an idea how to solve it.

[GeekMasher]

@geoffw0 it doesn't seem to have worked. The postgres tests are passing now but the file://:0:0:0:0 tests aren't. Anything I can do from my end?

[geoffw0]

Anything I can do from my end?

You can work around it by accepting the results from CI into your .expected file. That should allow us to merge this pull request.

[GeekMasher]

@geoffw0 Seems that worked! Let me know if you need any other changes before merging

[geoffw0]

Nope, all LGTM.