Skip to content

Conversation

@krngrover94
Copy link
Collaborator

  • Introduced registryStatus field in DbAction and ApiCollection to track MCP Registry status.
  • Implemented updateApiCollectionRegistryStatus method in ClientActor, DbActor, and DbLayer for updating registry status.
  • Added new action in struts.xml for updating API collection registry status.
  • Integrated MCP Registry checks in McpToolsSyncJobExecutor to update API collections based on registry availability.
  • Created McpRegistryUtils for handling MCP Registry queries and caching results.

- Introduced `registryStatus` field in `DbAction` and `ApiCollection` to track MCP Registry status.
- Implemented `updateApiCollectionRegistryStatus` method in `ClientActor`, `DbActor`, and `DbLayer` for updating registry status.
- Added new action in `struts.xml` for updating API collection registry status.
- Integrated MCP Registry checks in `McpToolsSyncJobExecutor` to update API collections based on registry availability.
- Created `McpRegistryUtils` for handling MCP Registry queries and caching results.
@krngrover94 krngrover94 requested a review from Ark2307 November 5, 2025 03:52
Copy link
Contributor

@devsecopsbot devsecopsbot bot left a comment

Choose a reason for hiding this comment

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

🤖 AI Security analysis: "Automated security scan found no issues in the changed files and no automatic fixes applied. No high‑risk findings detected; proceed with normal review and deployment."

Risk Level AI Score
🟢 NO RISK 2.0/100

Top 0 security issues / 0 total (Critical: 0, High: 0, Medium: 0, Low: 0)

Title Location Recommendation
No issues to display

🔗 View Detailed Report

private static final String ENV_ENABLE_REGISTRY_CHECK = "ENABLE_MCP_REGISTRY_CHECK";
private static final boolean ENABLE_REGISTRY_CHECK_DEFAULT = true;
private static final boolean enableRegistryCheck = Boolean.parseBoolean(
System.getenv().getOrDefault(ENV_ENABLE_REGISTRY_CHECK, String.valueOf(ENABLE_REGISTRY_CHECK_DEFAULT))
Copy link
Contributor

Choose a reason for hiding this comment

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

add try catch or normal simple check of value.equals(true) and use boolean so there is no chance of null value

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