Skip to content

fix: ensure openai 404 errors are retried - #10590

Merged
rekram1-node merged 1 commit into
anomalyco:devfrom
tim-smart:codex-retry
Jan 26, 2026
Merged

fix: ensure openai 404 errors are retried#10590
rekram1-node merged 1 commit into
anomalyco:devfrom
tim-smart:codex-retry

Conversation

@tim-smart

Copy link
Copy Markdown
Contributor

What does this PR do?

Closes #10505

How did you verify your code works?

Ran the dev build and testing it against the openai api

@github-actions

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

⚠️ Potential Duplicate Found:

This PR appears to be addressing the same issue as the current PR (#10590). Both are focused on ensuring OpenAI 404 errors are retried. You should verify if PR #10507 has already been merged or is still open, as it may make the current PR redundant.

@tim-smart

Copy link
Copy Markdown
Contributor Author

The following comment was made by an LLM, it may be inaccurate:

⚠️ Potential Duplicate Found:

This PR appears to be addressing the same issue as the current PR (#10590). Both are focused on ensuring OpenAI 404 errors are retried. You should verify if PR #10507 has already been merged or is still open, as it may make the current PR redundant.

The other PR does not fix the issue

@rekram1-node

Copy link
Copy Markdown
Collaborator

it seems weird to retry a 404, codex cli doesnt do this either afaik

@tim-smart

Copy link
Copy Markdown
Contributor Author

it seems weird to retry a 404, codex cli doesnt do this either afaik

Yes codex cli retries 404.

@rekram1-node

rekram1-node commented Jan 26, 2026

Copy link
Copy Markdown
Collaborator

Ig I didn't see it, looking again

@rekram1-node

Copy link
Copy Markdown
Collaborator

Ah i see it, weird

@rekram1-node
rekram1-node merged commit 837037c into anomalyco:dev Jan 26, 2026
4 checks passed
@rekram1-node

Copy link
Copy Markdown
Collaborator

Thanks!

@tim-smart

Copy link
Copy Markdown
Contributor Author

I guess ideally this would be up streamed to ai sdk

@rekram1-node

Copy link
Copy Markdown
Collaborator

yeah true

xenonwellz pushed a commit to xenonwellz/opencode that referenced this pull request Jan 28, 2026
Colin4k1024 pushed a commit to Colin4k1024/opencode that referenced this pull request Feb 10, 2026
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request May 5, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
avion23 pushed a commit to avion23/opencode that referenced this pull request Jun 10, 2026
Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request Jun 14, 2026
MarsQiu007 referenced this pull request in MarsQiu007/openNovel Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

If openai responds with a 404 status, retry the request

2 participants