Skip to content

Commit 24e1a6e

Browse files
handsomejack-42bgavrilMS
authored andcommitted
refactor: remove UserRealmURIPrefix from authority.Info
The field isn't used anywhere. Removing it simplifies the work on introducing authority abstraction. We might re-add it once we need it for anything.
1 parent eb50da8 commit 24e1a6e

File tree

2 files changed

+0
-3
lines changed

2 files changed

+0
-3
lines changed

apps/internal/oauth/ops/authority/authority.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -343,7 +343,6 @@ type Info struct {
343343
Host string
344344
CanonicalAuthorityURI string
345345
AuthorityType string
346-
UserRealmURIPrefix string
347346
ValidateAuthority bool
348347
Tenant string
349348
Region string
@@ -380,7 +379,6 @@ func NewInfoFromAuthorityURI(authority string, validateAuthority bool, instanceD
380379
Host: u.Host,
381380
CanonicalAuthorityURI: fmt.Sprintf("https://%v/%v/", u.Host, tenant),
382381
AuthorityType: authorityType,
383-
UserRealmURIPrefix: fmt.Sprintf("https://%v/common/userrealm/", u.Hostname()),
384382
ValidateAuthority: validateAuthority,
385383
Tenant: tenant,
386384
InstanceDiscoveryDisabled: instanceDiscoveryDisabled,

apps/internal/oauth/ops/authority/authority_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -305,7 +305,6 @@ func TestCreateAuthorityInfoFromAuthorityUri(t *testing.T) {
305305
Host: "login.microsoftonline.com",
306306
CanonicalAuthorityURI: authorityURI,
307307
AuthorityType: "MSSTS",
308-
UserRealmURIPrefix: "https://login.microsoftonline.com/common/userrealm/",
309308
Tenant: "common",
310309
ValidateAuthority: true,
311310
}

0 commit comments

Comments
 (0)