Repository navigation
Updated request and pagination parameters to reduce historical event … - #15223
Closed
v-krishnachi (v-krishnachi) wants to merge 2 commits into
Closed
v-krishnachi (v-krishnachi) wants to merge 2 commits into
v-krishnachi (v-krishnachi) wants to merge 2 commits into
Conversation
…polling and mitigate API rate limiting
Contributor
There was a problem hiding this comment.
Note
Copilot was unable to run its full agentic suite in this review.
Copilot review overview
Review effort: Lite
Findings: 1
Open (3)
The valueUnixTimestampInMillslooks like a typo (commonlyMillis/Milliseconds). If this is… · New Thelimitincrease to 500 (and matchingpageSize=500 below) changes the request shape and could… · New The PR description still contains the template placeholders (Change(s)/Reason/Testing/etc.) rather… · New
What changed in this PR
Updates the Atlassian Organization Audit connector polling configuration to reduce historical event polling and mitigate API rate limiting, plus associated version bumps.
Changes:
- Bumped solution and connector versions to 3.0.3 across package metadata.
- Updated poller request/pagination parameters (limit, from, pagination throttling, time format).
- Added a 3.0.3 entry to release notes documenting the change.
| File | Description |
|---|---|
| Solutions/AtlassianOrganizationAudit/ReleaseNotes.md | Adds release note entry for 3.0.3 describing the polling/pagination update. |
| Solutions/AtlassianOrganizationAudit/Package/mainTemplate.json | Bumps solution versions and updates poller query/pagination settings in the ARM template. |
| Solutions/AtlassianOrganizationAudit/Data/Solution_AtlassianOrganizationAudit.json | Updates solution and CCF connector version metadata to 3.0.3. |
| Solutions/AtlassianOrganizationAudit/Data Connectors/AtlassianOrganizationAuditConnector_CCF/AtlassianOrganizationAudit_PollerConfig.json | Updates the runtime poller config (throttle, time format, paging, query params). |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Contributor
Author
|
created new pull request as branch was behind the master and conflicts created lot additional files added. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Summary
Updates the Atlassian Organization Audit Events CCF connector to reduce historical backlog polling and mitigate Atlassian API rate limiting.
The connector previously used an unbounded persistent cursor with ascending sorting. For high-volume Atlassian organizations, a new connection could therefore paginate through a large historical backlog before reaching current events. Paginated requests were also not explicitly throttled, which could result in HTTP 429 responses.
Changes
fromquery parameter using CCF’s{_QueryWindowStartTime}value so new connections start from a recent time boundary.queryTimeFormat: UnixTimestampInMillsto format thefromvalue as Unix epoch milliseconds, as required by Atlassian.limitand CCFpageSizefrom 200 to 500, the documented maximum supported by/events-stream.pageSizeParameterName: limitto keep the CCF page size and API request parameter aligned.paginatedCallsPerSecond: 0.8, limiting pagination to approximately 48 requests per minute and keeping it below Atlassian’s documented limit of 50 requests per minute per API path.pagingQueryParamOnly: trueso subsequent cursor requests use the paging parameters without repeatedly applying the initial time boundary.PersistentTokenpagination so polling continues from the last saved Atlassian cursor across polling cycles.Package/mainTemplate.jsonand added the 3.0.3 solution package.Expected behavior
For a new or reset connection:
This reduces historical backlog processing and the likelihood of HTTP 429 responses. It also reduces the number of API calls required to retrieve the same event volume by up to 60% compared with the previous 200-event page size.
Customer impact
AtlassianAuditEvents_CLis unaffected.fromboundary to take effect.Documentation validation
UnixTimestampInMillsas a supportedQueryTimeFormatconstant. The spelling is intentional./events-streamfromparameter as Unix epoch time in milliseconds./events-streamlimitrange as 1–500, with 500 as the maximum.References:
/events-streamAPI: https://developer.atlassian.com/cloud/admin/organization/rest/api-group-events/#api-v1-orgs-orgid-events-stream-getValidation performed
git diff --checkpassed.Validation summary: 11 passed, 0 failed, 6 skipped. The skipped checks were not applicable or unavailable in the local environment (
.NET Core 3.1and TruffleHog were not installed).Files changed
Solutions/AtlassianOrganizationAudit/Data Connectors/AtlassianOrganizationAuditConnector_CCF/AtlassianOrganizationAudit_PollerConfig.jsonSolutions/AtlassianOrganizationAudit/Data/Solution_AtlassianOrganizationAudit.jsonSolutions/AtlassianOrganizationAudit/Package/mainTemplate.jsonSolutions/AtlassianOrganizationAudit/Package/3.0.3.zipSolutions/AtlassianOrganizationAudit/ReleaseNotes.md