Skip to content

Test fix vulnerability with linting - #319

Merged
cjbarth merged 2 commits into
node-saml:4.xfrom
jindazhao01:test-fix-vulnerability
Aug 19, 2023
Merged

Test fix vulnerability with linting#319
cjbarth merged 2 commits into
node-saml:4.xfrom
jindazhao01:test-fix-vulnerability

Conversation

@jindazhao01

@jindazhao01 jindazhao01 commented Aug 18, 2023

Copy link
Copy Markdown
Contributor

Description

Deleted duplicated test in validateRedirect() to fix test issue

210 passing (561ms)

@jindazhao01

Copy link
Copy Markdown
Contributor Author

@cjbarth @markstos not sure why I was denied permissions to push to the repo itself, but tagging you here as this is the fix

@cjbarth
cjbarth merged commit f35191d into node-saml:4.x Aug 19, 2023
@markstos

Copy link
Copy Markdown

@jindazhao01 Only committers can push to the main repo.

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.

4 participants