Skip to content

fix: Add missing content-type#768

Merged
tevincent merged 1 commit into
mainfrom
fix-2fa-content-type
May 19, 2026
Merged

fix: Add missing content-type#768
tevincent merged 1 commit into
mainfrom
fix-2fa-content-type

Conversation

@tevincent
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 18, 2026 13:53
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an explicit JSON Content-Type header to two Ktor POST requests so the backend reliably receives the payload as JSON (and Ktor’s serialization/content negotiation behaves consistently).

Changes:

  • Set contentType(ContentType.Application.Json) for the notifications device registration POST.
  • Set contentType(ContentType.Application.Json) for the cross-app device info update POST.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Notifications/Registration/src/main/kotlin/AbstractNotificationsRegistrationWorker.kt Adds explicit JSON Content-Type to the device registration POST request.
CrossAppLogin/Back/src/main/kotlin/com/infomaniak/core/crossapplogin/back/internal/deviceinfo/AbstractDeviceInfoUpdateWorker.kt Adds explicit JSON Content-Type to the device info update POST request.

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

@tevincent tevincent merged commit 3a31ddd into main May 19, 2026
12 checks passed
@tevincent tevincent deleted the fix-2fa-content-type branch May 19, 2026 06: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