Skip to content

Change Tesla API to TLS 1.3 - openWB issue #3439#3582

Merged
LKuemmel merged 3 commits into
openWB:masterfrom
admontis:master
Jun 24, 2026
Merged

Change Tesla API to TLS 1.3 - openWB issue #3439#3582
LKuemmel merged 3 commits into
openWB:masterfrom
admontis:master

Conversation

@admontis

Copy link
Copy Markdown
Contributor

This PR changes the Tesla API to use TLS1.3. It is required to be able to query the owner-api.

@Stabilostick

Copy link
Copy Markdown

Ich vote für eine Übernahme zum Testen.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Tesla vehicle integration to force TLS 1.3 when calling Tesla’s owner-api and authentication endpoints, addressing connectivity issues described in openWB issue #3439.

Changes:

  • Introduces a custom requests HTTPAdapter that restricts TLS negotiation to TLS 1.3.
  • Uses a Tesla-specific HTTP session that mounts this adapter for owner-api.teslamotors.com and auth.tesla.com.
  • Routes Tesla API calls (wake-up, token refresh, data requests) through the new TLS 1.3 session.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/modules/vehicles/tesla/api.py
Comment thread packages/modules/vehicles/tesla/api.py Outdated
Comment thread packages/modules/vehicles/tesla/api.py Outdated
Comment thread packages/modules/vehicles/tesla/api.py Outdated
Comment thread packages/modules/vehicles/tesla/api.py
@LKuemmel

Copy link
Copy Markdown
Contributor

Bitte noch die Hinweise von Copilot beachten.

@admontis

Copy link
Copy Markdown
Contributor Author

CoPilot review ist eingearbeitet

@LKuemmel LKuemmel linked an issue Jun 24, 2026 that may be closed by this pull request
@LKuemmel LKuemmel merged commit c5b873e into openWB:master Jun 24, 2026
1 check passed
@Stabilostick

Copy link
Copy Markdown

Tesla-Auth geht bei mir damit.

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.

4 participants