Skip to content

KNOX-2070 - SSOCookieFederationFilter NPE#170

Merged
risdenk merged 1 commit intoapache:masterfrom
risdenk:KNOX-2070
Oct 29, 2019
Merged

KNOX-2070 - SSOCookieFederationFilter NPE#170
risdenk merged 1 commit intoapache:masterfrom
risdenk:KNOX-2070

Conversation

@risdenk
Copy link
Copy Markdown
Contributor

@risdenk risdenk commented Oct 28, 2019

What changes were proposed in this pull request?

Add test for XHR and fix the NPE caused by XHR and invalid cookies.

How was this patch tested?

mvn -T.5C clean verify -Ppackage,release -Dshellcheck

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

Signed-off-by: Kevin Risden <krisden@apache.org>
@risdenk risdenk merged commit b3923f2 into apache:master Oct 29, 2019
@risdenk risdenk deleted the KNOX-2070 branch October 29, 2019 16:33
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
Signed-off-by: Kevin Risden <krisden@apache.org>
Signed-off-by: Kevin Risden <krisden@cloudera.com>
Change-Id: Iaee58fa1bc7d4d3ff685bc63f3b2c41672309282
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
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.

2 participants