Skip to content

fix: Add utility for handling localhost certificate in debug mode - #834

Merged
tevincent merged 4 commits into
mainfrom
fix-localhot-certificate
Aug 14, 2026
Merged

fix: Add utility for handling localhost certificate in debug mode#834
tevincent merged 4 commits into
mainfrom
fix-localhot-certificate

Conversation

@sirambd

@sirambd sirambd commented Aug 14, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings August 14, 2026 12:50

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

Restricts localhost SSL certificate bypasses to debuggable applications.

Changes:

  • Extracts localhost certificate validation into a helper.
  • Checks the host application's debuggable flag before bypassing SSL errors.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Login/src/main/java/com/infomaniak/core/login/LoginWebViewClient.kt Outdated
@sonarqubecloud

Copy link
Copy Markdown

@tevincent
tevincent enabled auto-merge August 14, 2026 13:24
@tevincent
tevincent merged commit 1c683a3 into main Aug 14, 2026
9 checks passed
@tevincent
tevincent deleted the fix-localhot-certificate branch August 14, 2026 13:25
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.

3 participants