Skip to content

Commit

Permalink
refactor: remove UserRealmURIPrefix from authority.Info
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
handsomejack-42 committed May 9, 2024
1 parent ae2db6b commit 9575c6a
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
2 changes: 0 additions & 2 deletions apps/internal/oauth/ops/authority/authority.go
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,6 @@ type Info struct {
Host string
CanonicalAuthorityURI string
AuthorityType string
UserRealmURIPrefix string
ValidateAuthority bool
Tenant string
Region string
Expand Down Expand Up @@ -381,7 +380,6 @@ func NewInfoFromAuthorityURI(authority string, validateAuthority bool, instanceD
Host: u.Host,
CanonicalAuthorityURI: fmt.Sprintf("https://%v/%v/", u.Host, tenant),
AuthorityType: authorityType,
UserRealmURIPrefix: fmt.Sprintf("https://%v/common/userrealm/", u.Hostname()),
ValidateAuthority: validateAuthority,
Tenant: tenant,
InstanceDiscoveryDisabled: instanceDiscoveryDisabled,
Expand Down
1 change: 0 additions & 1 deletion apps/internal/oauth/ops/authority/authority_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,6 @@ func TestCreateAuthorityInfoFromAuthorityUri(t *testing.T) {
Host: "login.microsoftonline.com",
CanonicalAuthorityURI: authorityURI,
AuthorityType: "MSSTS",
UserRealmURIPrefix: "https://login.microsoftonline.com/common/userrealm/",
Tenant: "common",
ValidateAuthority: true,
}
Expand Down

0 comments on commit 9575c6a

Please sign in to comment.