Skip to content

Incorrect authority parsing in MergedOptions.ParseAuthorityIfNecessary #3262

@JoshLozensky

Description

@JoshLozensky

Microsoft.Identity.Web Library

Microsoft.Identity.Web

Microsoft.Identity.Web version

3.7.1

Web app

Not Applicable

Web API

Not Applicable

Token cache serialization

Not Applicable

Description

A bug introduced in 3.1.0 is during splitting of the authority.
The condition will always be true here

This was originally brought up by @yzhoholiev in another issue but isn't relevant to the original post so creating this to track it.

Reproduction steps

Use a config that needs authority parsing

Error message

No response

Id Web logs

No response

Relevant code snippets

N/A

Regression

No response

Expected behavior

This condition should actually switch on the presence of tenantId/instance rather than be always true

Metadata

Metadata

Assignees

Labels

P2bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions