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

Publish STATUS_MESSAGE & add GetInvocationData siddhi app #5

Merged
merged 4 commits into from
Sep 18, 2024

Conversation

anjuchamantha
Copy link
Contributor

Publish STATUS_MESSAGE & add GetInvocationData siddhi app

With this PR,

  1. STATUS_MESSAGE is published to API_INVOCATION_RAW_DATA table. This new table column is added via APIRawDataSubmission siddhi app.
  2. GetInvocationData siddhi app is added to get the invocation data with http://localhost:8007/InvocationData/StatusCheckStream

Note:

Need to add statusMessage as a stream attribute with priority 7 in both IS and APIM deployment.toml files. And change the priority of other attributes after 7.

[[open_banking.data_publishing.thrift.stream.attributes]]
name="statusMessage"
priority=7

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).

Automation Test Details

Test Suite Test Script IDs
Integration Suite TCXXXXX, TCXXXX

Conformance Tests Details

Test Suite Name Test Suite Version Scenarios Result
Security Suite VX.X Foo, Bar Passed

Resources

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

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

@anjuchamantha anjuchamantha changed the title [WIP] Publish STATUS_MESSAGE & add GetInvocationData siddhi app Publish STATUS_MESSAGE & add GetInvocationData siddhi app Sep 10, 2024
aka4rKO
aka4rKO previously approved these changes Sep 13, 2024
@imesh94 imesh94 merged commit 80c471c into wso2:main Sep 18, 2024
1 check passed
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