Skip to content

Set deepl auth header#16906

Merged
jfaltermeier merged 1 commit intomasterfrom
msujew/fix-deepl-auth
Jan 29, 2026
Merged

Set deepl auth header#16906
jfaltermeier merged 1 commit intomasterfrom
msujew/fix-deepl-auth

Conversation

@msujew
Copy link
Copy Markdown
Member

@msujew msujew commented Jan 29, 2026

What it does

Fixes the recent 403 translation errors. It looks like DeepL no longer requires to set the key in the body, but instead in the request header (which makes more sense to be honest). I'll clean up the deepl request code in a separate PR. This is just to fix the immediate issue.

How to test

Run the localization script locally. It should work as expected (after setting a key).

Review checklist

Reminder for reviewers

@msujew msujew requested a review from jfaltermeier January 29, 2026 14:13
@github-project-automation github-project-automation Bot moved this to Waiting on reviewers in PR Backlog Jan 29, 2026
Copy link
Copy Markdown
Contributor

@jfaltermeier jfaltermeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, lgtm.
I triggered a run from this branch and it worked.
https://github.com/eclipse-theia/theia/actions/runs/21481567638/job/61879025234
#16907

@github-project-automation github-project-automation Bot moved this from Waiting on reviewers to Needs merge in PR Backlog Jan 29, 2026
@jfaltermeier jfaltermeier merged commit 36a3959 into master Jan 29, 2026
11 checks passed
@jfaltermeier jfaltermeier deleted the msujew/fix-deepl-auth branch January 29, 2026 14:38
@github-project-automation github-project-automation Bot moved this from Needs merge to Done in PR Backlog Jan 29, 2026
@github-actions github-actions Bot added this to the 1.68.0 milestone Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants