add an experimental gemini call to the backend #2974
Conversation
|
Looks like we need to fiddle with the package versions on main: https://github.com/dart-lang/dart-pad/actions/runs/9354896383/job/25748806872?pr=2974 |
| } | ||
|
|
||
| // Only allow the call from known clients / endpoints. | ||
| const firebaseHostAddress = '199.36.158.100'; |
There was a problem hiding this comment.
Would this be the address used if calling from a Firebase-hosted cloud function or similar?
There was a problem hiding this comment.
Hmm, I don't know.
This IP is often (always?) the address for firebase hosted web apps (though, this may or may not always be true with custom domains).
Fair to say that this check has some room for improvement.
There was a problem hiding this comment.
An update here - after an in-person discussion w/ @johnpryan, I switched to checking that the request origin is https://dartpad.dev.
Co-authored-by: Parker Lougheed <[email protected]>
There was a problem hiding this comment.
As long as any client calling the API as many times as they'd like is not a concern, looks good to me.
We could put some call throttling in perhaps. I'm not committed to this particular impl., but do like the idea of experimenting against a live backend. If we see usage issues (way too much, ...) we can always remove the API / otherwise pivot.
| } | ||
|
|
||
| // Only allow the call from known clients / endpoints. | ||
| const firebaseHostAddress = '199.36.158.100'; |
There was a problem hiding this comment.
Hmm, I don't know.
This IP is often (always?) the address for firebase hosted web apps (though, this may or may not always be true with custom domains).
Fair to say that this check has some room for improvement.
Add an experimental gemini call to the backend:
GOOGLE_API_KEYwill need to be configured in the cloud serving infragemini-1.5-flash-latestmodelContribution guidelines:
dart format.Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.