Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Azure SDK API - IWA specific call returns 415 on MSAL but works on ADAL #2193

Closed
1 of 7 tasks
bgavrilMS opened this issue Nov 17, 2020 · 4 comments
Closed
1 of 7 tasks
Assignees
Labels
bug external P2 regression Behavior that worked in a previous release that no longer works in a newer release requires more info

Comments

@bgavrilMS
Copy link
Member

Which Version of MSAL are you using ?
MSAL 4.22

Platform
net fwk

What authentication flow has the issue?

  • Desktop / Mobile
    • Interactive
    • Integrated Windows Auth
    • Username Password
    • Device code flow (browserless)
  • Web App
    • Authorization code
    • OBO
  • Daemon App
    • Service to Service calls

Other? - please describe;

Is this a new or existing app?
PowerShell

Repro

POST /adfs/services/trust/2005/usernamemixed fails with

HTTP/1.1 415 Cannot process the message because the content type 'application/octet-stream' was not the expected type 'application/soap+xml; charset=utf-8'.
Content-Length: 0
Content-Type: text/html; charset=utf-8
Server: Microsoft-HTTPAPI/2.0 Microsoft-HTTPAPI/2.0
Date: Tue, 17 Nov 2020 11:10:36 GMT

This works on ADAL

Note: I have network traces but will not post them here for PII reasons.

@bgavrilMS bgavrilMS added bug scenario:IWA P1 regression Behavior that worked in a previous release that no longer works in a newer release labels Nov 17, 2020
@bgavrilMS bgavrilMS self-assigned this Nov 17, 2020
@bgavrilMS
Copy link
Member Author

P1 as this regresses ADAL functionality and is affecting users in production.

@bgavrilMS bgavrilMS added this to Todo/Committed in MSAL.NET (legacy) via automation Nov 19, 2020
@bgavrilMS bgavrilMS moved this from Todo/Committed to In Progress in MSAL.NET (legacy) Nov 19, 2020
@bgavrilMS bgavrilMS added this to the 4.23.0 milestone Nov 19, 2020
@bgavrilMS bgavrilMS moved this from In Progress to Blocked/Waiting for reply in MSAL.NET (legacy) Nov 20, 2020
@bgavrilMS
Copy link
Member Author

Root cause seems to be a missing header Content-Type: application/soap+xml but MSAL clearly sets it

@bgavrilMS
Copy link
Member Author

Ack @henrik-me l, I'll focus on this

@henrik-me henrik-me added P2 and removed P1 labels Dec 17, 2020
@bgavrilMS
Copy link
Member Author

Turns out this is external and was fixed by Azure SDK

@bgavrilMS bgavrilMS removed this from Blocked/Waiting for reply in MSAL.NET (legacy) Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug external P2 regression Behavior that worked in a previous release that no longer works in a newer release requires more info
Projects
None yet
Development

No branches or pull requests

2 participants