Skip to content

KNOX-2389 - AliasBasedTokenStateService stops processing persisted jo…#346

Merged
pzampino merged 1 commit intoapache:masterfrom
pzampino:master
Jun 16, 2020
Merged

KNOX-2389 - AliasBasedTokenStateService stops processing persisted jo…#346
pzampino merged 1 commit intoapache:masterfrom
pzampino:master

Conversation

@pzampino
Copy link
Copy Markdown
Contributor

…urnal entries if one is malformed

What changes were proposed in this pull request?

Added some defensive logic to prevent skipping processing valid token state journal entries due one or more invalid entries.

How was this patch tested?

mvn -T1.5C -Ppackage,release clean install, augmented AliasBasedTokenStateServiceTest to include a specific test for this, and performed manual testing.

@pzampino pzampino requested a review from smolnar82 June 15, 2020 21:39
Copy link
Copy Markdown
Contributor

@smolnar82 smolnar82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@moresandeep moresandeep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@pzampino pzampino merged commit dcdeace into apache:master Jun 16, 2020
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
…urnal entries if one is malformed (apache#346)

Change-Id: I724b6f6a226d518be2ce601bfccdd555ebbec452
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
…sted journal entries if one is malformed (apache#346)" into cdpd-master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants