Skip to content

Commit

Permalink
Change identifier to linkID (#186)
Browse files Browse the repository at this point in the history
* Change identifier to linkID fixes descope/etc#1148 + test

* Update descope/auth/types.go

* Fix comment

Co-authored-by: Asaf Shen <[email protected]>
  • Loading branch information
aviadl and asafshen authored Dec 10, 2022
1 parent deeb850 commit 2822c1a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
16 changes: 8 additions & 8 deletions descope/auth/enchantedlink_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,14 +47,14 @@ func TestSignInEnchantedLink(t *testing.T) {
assert.EqualValues(t, uri, m["URI"])
return &http.Response{
StatusCode: http.StatusOK,
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s","identifier": "%s"}`, pendingRefResponse, identifier))),
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s","linkId": "%s"}`, pendingRefResponse, identifier))),
}, nil
})
require.NoError(t, err)
response, err := a.EnchantedLink().SignIn(email, uri, nil, nil)
require.NoError(t, err)
require.Equal(t, pendingRefResponse, response.PendingRef)
require.Equal(t, identifier, response.Identifier)
require.Equal(t, identifier, response.LinkID)
}

func TestSignInEnchantedLinkStepup(t *testing.T) {
Expand All @@ -79,14 +79,14 @@ func TestSignInEnchantedLinkStepup(t *testing.T) {
assert.EqualValues(t, "test", bearers[1])
return &http.Response{
StatusCode: http.StatusOK,
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s","identifier": "%s"}`, pendingRefResponse, identifier))),
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s","linkId": "%s"}`, pendingRefResponse, identifier))),
}, nil
})
require.NoError(t, err)
response, err := a.EnchantedLink().SignIn(email, uri, &http.Request{Header: http.Header{"Cookie": []string{"DSR=test"}}}, &LoginOptions{Stepup: true, CustomClaims: map[string]interface{}{"k1": "v1"}})
require.NoError(t, err)
require.Equal(t, pendingRefResponse, response.PendingRef)
require.Equal(t, identifier, response.Identifier)
require.Equal(t, identifier, response.LinkID)
}

func TestSignInEnchantedLinkInvalidResponse(t *testing.T) {
Expand Down Expand Up @@ -120,14 +120,14 @@ func TestSignUpEnchantedLink(t *testing.T) {
assert.EqualValues(t, "test", m["user"].(map[string]interface{})["name"])
return &http.Response{
StatusCode: http.StatusOK,
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s","identifier": "%s"}`, pendingRefResponse, identifier))),
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s","linkId": "%s"}`, pendingRefResponse, identifier))),
}, nil
})
require.NoError(t, err)
response, err := a.EnchantedLink().SignUp(email, uri, &User{Name: "test"})
require.NoError(t, err)
require.Equal(t, pendingRefResponse, response.PendingRef)
require.Equal(t, identifier, response.Identifier)
require.Equal(t, identifier, response.LinkID)
}

func TestSignUpOrInEnchantedLink(t *testing.T) {
Expand All @@ -144,14 +144,14 @@ func TestSignUpOrInEnchantedLink(t *testing.T) {
assert.EqualValues(t, uri, m["URI"])
return &http.Response{
StatusCode: http.StatusOK,
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s", "identifier": "%s"}`, pendingRefResponse, identifier))),
Body: io.NopCloser(bytes.NewBufferString(fmt.Sprintf(`{"pendingRef": "%s", "linkId": "%s"}`, pendingRefResponse, identifier))),
}, nil
})
require.NoError(t, err)
response, err := a.EnchantedLink().SignUpOrIn(email, uri)
require.NoError(t, err)
require.Equal(t, pendingRefResponse, response.PendingRef)
require.Equal(t, identifier, response.Identifier)
require.Equal(t, identifier, response.LinkID)
}

func TestSignUpEnchantedLinkEmptyIdentifier(t *testing.T) {
Expand Down
2 changes: 1 addition & 1 deletion descope/auth/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ type JWTResponse struct {

type EnchantedLinkResponse struct {
PendingRef string `json:"pendingRef,omitempty"` // Pending referral code used to poll enchanted link authentication status
Identifier string `json:"identifier,omitempty"` // Identifier, on which link the user should click
LinkID string `json:"linkId,omitempty"` // Link id, on which link the user should click
}

func NewAuthenticationInfo(jRes *JWTResponse, token *Token) *AuthenticationInfo {
Expand Down

0 comments on commit 2822c1a

Please sign in to comment.