Skip to content

Commit

Permalink
set both access and value modified values
Browse files Browse the repository at this point in the history
  • Loading branch information
korsvanloon committed Feb 20, 2024
1 parent 5107a61 commit ddfb793
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 8 deletions.
4 changes: 4 additions & 0 deletions src/token.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,10 @@ describe("FederatedToken", () => {
federatedToken.isAccessTokenModified(),
"isModified should be true when trackModified is true and token is modified"
);
assert.isTrue(
federatedToken.isValueModified(),
"isModified should be true when trackModified is true and value is modified"
);
});

test("serializeAccessToken", () => {
Expand Down
14 changes: 6 additions & 8 deletions src/token.ts
Original file line number Diff line number Diff line change
Expand Up @@ -93,15 +93,13 @@ export class FederatedToken {
Buffer.from(at, "base64").toString("ascii")
);

// The value is modified if:
this._valueModified =
this._valueModified = trackModified && !isEqual(this.values, token.values);

this._accessTokenModified =
trackModified &&
// either a value has been modified
(!isEqual(this.values, token.values) ||
// or at least 1 token has been modified
Object.keys(token.tokens).some(
(key) => !isEqual(this.tokens[key], token.tokens[key])
));
Object.keys(token.tokens).some(
(key) => !isEqual(this.tokens[key], token.tokens[key])
);

// Merge tokens and values into "this" object.
this.tokens = {
Expand Down

0 comments on commit ddfb793

Please sign in to comment.