Skip to content
This repository was archived by the owner on Jan 2, 2025. It is now read-only.

Log message with context for Quota/Answer API call failures #1135

Merged
ggordonhall merged 1 commit into from
Nov 16, 2023
Merged

Log message with context for Quota/Answer API call failures #1135

ggordonhall merged 1 commit into from
Nov 16, 2023

Conversation

@calyptobai
Copy link
Contributor

@calyptobai calyptobai commented Nov 15, 2023

Now, we correctly propagate errors up when either:

  • The LLM gateway fails to open a stream
  • The quota API returns unexpected JSON content
    • In this case, the original body is forwarded in an error for further examination

Closes BLO-1599

@ggordonhall ggordonhall self-requested a review November 16, 2023 09:31
@ggordonhall ggordonhall merged commit 5071ea5 into main Nov 16, 2023
@ggordonhall ggordonhall deleted the log-quota-answer-errors branch November 16, 2023 11:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@ggordonhall ggordonhall ggordonhall approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@calyptobai @ggordonhall