Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds testcase to check CORS for token endpoints #13241

Closed
wants to merge 6 commits into from

Conversation

Kirishikesan
Copy link
Contributor

Overview

This testcase checks whether the CORS valve integrated through [1] is working as expected through the following tests.

  1. Adds CORS config to deployment.toml
  2. Checks the CORS valve response by adding an allowed Origin header and a not allowed Origin header in the request
  3. Response should have the Access-Control-Allow-Origin header and a status code of 200 if the request is allowed and a 403 status code if the request is not allowed through CORS.

[1] wso2/api-manager#1175

@Kirishikesan Kirishikesan requested a review from chanikag July 10, 2023 06:58
@Kirishikesan Kirishikesan requested a review from chanikag July 18, 2023 10:37
import org.apache.commons.logging.LogFactory;

import org.testng.Assert;
import org.testng.annotations.*;
Copy link
Contributor

Choose a reason for hiding this comment

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

Avoid import all

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