feat: unified ingestLogs RPC with session or API key auth - #19
Merged
Conversation
Remove ingestLogsV1 and /v1/ingest direct calls; use ingestLogs RPC with bearer from authToken or apiKey. Point CloudWatch setup at /v1/ingestLogs.
Use authToken for typed RPC calls and apiKey for ingest only. Restore non-local logfox run by sending app name when no local app record exists.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ingestLogsV1and direct/v1/ingestfetch; use typed RPCingestLogsat/v1/ingestLogsauthToken; ingest prefersapiKeythen falls back toauthTokenlogfox runsends app UUID locally and app name for non-local API-key-only flows/v1/ingestLogsCompanion api-service PR: logfoxai/api-service (feat/dual-auth-request-context)
Test plan
npm run validate(build)logfox run --name my-app -- echo hellowith session auth