DomainTools CCF 3.0.1: sessionID/top params, rate limit - #15151
Conversation
There was a problem hiding this comment.
Note
Copilot was unable to run its full agentic suite in this review.
Pull request overview
Updates the DomainTools CCF solution to v3.0.1, adding session-based polling inputs and NDJSON streaming support while adjusting connector throttling behavior to better align with DomainTools API limits.
Changes:
- Bumped solution/package versioning and updated release notes to 3.0.1.
- Added customer-configurable parameters (feedTypes, sessionID, queryWindowInMin, optional top) and introduced per-feed conditional deployment.
- Switched feed responses to NDJSON (
jsonl) and added 429/Retry-After-aware rate limit handling.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| Solutions/DomainTools CCF/ReleaseNotes.md | Documents the 3.0.1 changes (new params, NDJSON, rate limiting). |
| Solutions/DomainTools CCF/Package/mainTemplate.json | Bumps version, adds new parameters/UI wiring, per-feed connector conditions, NDJSON response format, and rate limit config. |
| Solutions/DomainTools CCF/Package/createUiDefinition.json | Updates a user-facing description string for an analytic item. |
| Solutions/DomainTools CCF/Data/Solution_DomainTools.json | Bumps solution version to 3.0.1. |
| Solutions/DomainTools CCF/Data Connectors/DomainToolsFeedsLogs_ccf/DomainToolsFeedsLogs_connectorDefinition.json | Exposes new customer-configurable parameters in the connector UI step. |
| Solutions/DomainTools CCF/Data Connectors/DomainToolsFeedsLogs_ccf/DomainToolsFeedsLogs_PollerConfig.json | Adds per-feed conditions, NDJSON response format, and 429-aware rate limit config. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Hi v-atulyadav, Please review and merge. |
|
Hi Rambatla Venkat Rao (@RamboV) Please add the latest version at the top of the Release Notes and keep the comments concise. Thanks! |
|
I have updated the Release Notes. Please review. |




Change(s):
Solutions/DomainTools CCF/ReleaseNotes.md.sessionIDand an optionaltopparameter in:Solutions/DomainTools CCF/Data Connectors/DomainToolsFeedsLogs_ccf/DomainToolsFeedsLogs_connectorDefinition.jsonSolutions/DomainTools CCF/Data Connectors/DomainToolsFeedsLogs_ccf/DomainToolsFeedsLogs_PollerConfig.jsonSolutions/DomainTools CCF/Package/mainTemplate.jsonafter,before,fromBeginning) internal;afteris no longer passed to the API.rateLimitQPSto 1 and addedrateLimitConfigto enforce DomainTools' 2 requests/minute limit with 429/Retry-After handling.Accept: application/x-ndjson,response.format: jsonl) for efficient streaming.Solutions/DomainTools CCF/Package/3.0.1.zipwith the updated mainTemplate.json and createUiDefinition.json.Reason for Change(s):
RestApiPollerconnection rules.Retry-Afterheaders.Version Updated:
Solutions/DomainTools CCF/SolutionMetadata.json,Solutions/DomainTools CCF/Data/Solution_DomainTools.json, and the package.Testing Completed:
Checked that the validations are passing and have addressed any issues that are present: