Skip to content

Commit 13a2c42

Browse files
tolik0agarctfilleadbetdevin-ai-integration[bot]
authored
fix(source-intercom): Block simultaneous reading from companies endpoint (#71141)
Co-authored-by: Alfredo Garcia <alfredo.garcia@hallmark.edu> Co-authored-by: Lucas Leadbetter <5595530+lleadbet@users.noreply.github.com> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: alfredo.garcia@airbyte.io
1 parent fe9d01b commit 13a2c42

6 files changed

Lines changed: 186 additions & 228 deletions

File tree

airbyte-integrations/connectors/source-intercom/manifest.yaml

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,13 @@ check:
77
stream_names:
88
- tags
99

10+
stream_groups:
11+
companies_group:
12+
streams:
13+
- $ref: "#/definitions/streams/companies"
14+
action:
15+
type: BlockSimultaneousSyncsAction
16+
1017
definitions:
1118
streams:
1219
activity_logs:
@@ -263,6 +270,9 @@ definitions:
263270
Intercom-Version: "2.11"
264271
error_handler:
265272
type: DefaultErrorHandler
273+
backoff_strategies:
274+
- type: ConstantBackoffStrategy
275+
backoff_time_in_seconds: 60
266276
response_filters:
267277
- type: HttpResponseFilter
268278
http_codes:
@@ -281,11 +291,11 @@ definitions:
281291
- type: HttpResponseFilter
282292
http_codes:
283293
- 400
284-
action: FAIL
294+
action: RETRY
285295
failure_type: transient_error
286296
error_message: >-
287-
Scroll already exists for this workspace. Please ensure you do not have multiple syncs running at the same time.
288-
Intercom API does not allow for you to run the Companies or Company Parts streams in parallel.
297+
Scroll already exists for this workspace. Retrying after backoff.
298+
Intercom API does not allow more than one scroll open per app at a time.
289299
- type: HttpResponseFilter
290300
http_codes:
291301
- 500

airbyte-integrations/connectors/source-intercom/metadata.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@ data:
66
hosts:
77
- api.intercom.io
88
connectorBuildOptions:
9-
baseImage: docker.io/airbyte/source-declarative-manifest:7.6.1@sha256:b433ec72f88e69bd836f3c7c815c21bfeeeb32501aeabf586bc421875289e5e2
9+
baseImage: docker.io/airbyte/source-declarative-manifest:7.6.1.post28.dev22724700845@sha256:ac9cce26e99ba3f0848d5c10b566795d1400ccf943b993249ead41c30facbf2c
1010
connectorSubtype: api
1111
connectorType: source
1212
definitionId: d8313939-3782-41b0-be29-b3ca20d8dd3a
13-
dockerImageTag: 0.13.16-rc.4
13+
dockerImageTag: 0.13.16-rc.5
1414
dockerRepository: airbyte/source-intercom
1515
documentationUrl: https://docs.airbyte.com/integrations/sources/intercom
1616
externalDocumentationUrls:
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# Override the default manifest-only connector tasks to pin the CDK CLI
2+
# to the same dev version used by the connector's base image.
3+
# This is needed because the connector uses features (stream_groups)
4+
# that are not yet available in the latest stable CDK release.
5+
include = [
6+
"${POE_GIT_DIR}/poe-tasks/manifest-only-connector-tasks.toml",
7+
]
8+
9+
[tasks]
10+
install-cdk-cli = "uv tool install --upgrade 'airbyte-cdk[dev]==7.6.1.post28.dev22724700845'"

0 commit comments

Comments
 (0)