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

Added separate executor list to handle CDS unauthenticated endpoints #19

Merged
merged 5 commits into from
Oct 21, 2024

Conversation

aka4rKO
Copy link
Contributor

@aka4rKO aka4rKO commented Oct 16, 2024

Added separate executor list to handle CDS unauthenticated endpoints

$subject by not engaging the MTLS executor for the following unauthenticated elected resources:
"/banking/products", "/banking/products/{productId}", "/discovery/status", "/discovery/outages"

Issue link:

Doc Issue: Optional, link issue from documentation repository

Applicable Labels: OB 3.0.0, CDS Toolkit


Development Checklist

  1. Built complete solution with pull request in place.
  2. Ran checkstyle plugin with pull request in place.
  3. Ran Findbugs plugin with pull request in place.
  4. Ran FindSecurityBugs plugin and verified report.
  5. Formatted code according to WSO2 code style.
  6. Have you verify the PR does't commit any keys, passwords, tokens, usernames, or other secrets?
  7. Migration scripts written (if applicable).
  8. Have you followed secure coding standards in WSO2 Secure Engineering Guidelines?

Testing Checklist

  1. Written unit tests.
  2. Documented test scenarios(link available in guides).
  3. Written automation tests (link available in guides).
  4. Verified tests in multiple database environments (if applicable).
  5. Verified tests in multiple deployed specifications (if applicable).
  6. Tested with OBBI enabled (if applicable).
  7. Tested with specification regulatory conformance suites (if applicable).

Resources

Knowledge Base: https://sites.google.com/wso2.com/open-banking/

Guides: https://sites.google.com/wso2.com/open-banking/developer-guides

@imesh94
Copy link
Contributor

imesh94 commented Oct 18, 2024

Shall we change the logic of detecting Common endpoints along with this?

@aka4rKO
Copy link
Contributor Author

aka4rKO commented Oct 18, 2024

Shall we change the logic of detecting Common endpoints along with this?

@imesh94 done with c334acc

imesh94
imesh94 previously approved these changes Oct 18, 2024
@aka4rKO aka4rKO added the Ready-to-Build Adding this label will trigger a new build job. label Oct 21, 2024
@imesh94 imesh94 merged commit 9a29d3a into wso2:main Oct 21, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready-to-Build Adding this label will trigger a new build job.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants