Skip to content

fix: snowpipe streaming remove authz error handling#6777

Merged
achettyiitr merged 8 commits into
release/1.70.xfrom
fix.snowpipe-streaming-remove-authz
Mar 19, 2026
Merged

fix: snowpipe streaming remove authz error handling#6777
achettyiitr merged 8 commits into
release/1.70.xfrom
fix.snowpipe-streaming-remove-authz

Conversation

@achettyiitr

@achettyiitr achettyiitr commented Mar 17, 2026

Copy link
Copy Markdown
Member

🔒 Scanned for secrets using gitleaks 8.29.1

Description

  • Remove authz error handling for snowpipe streaming

Linear Ticket

  • Resolves INT-6090

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

atzoum and others added 2 commits March 17, 2026 10:16
@atzoum
chore: sync release v1.70.0 to main branch (#6775)
74125a6 
# Description

Syncing release v1.70.0 to main branch

**WARNING: Do NOT rewrite git history and ALWAYS use a "Merge Commit"
for merging!**

**↓↓ Please review and edit commit overrides before merging ↓↓**

BEGIN_COMMIT_OVERRIDE

END_COMMIT_OVERRIDE
@achettyiitr
fix: snowpipe streaming remove authz error handling
c23d808 
🔒 Scanned for secrets using gitleaks 8.29.1
@achettyiitr achettyiitr force-pushed the fix.snowpipe-streaming-remove-authz branch from f88e9be to c23d808 Compare March 17, 2026 19:09
atzoum added 2 commits March 18, 2026 10:30
@atzoum @achettyiitr
chore: bump rudder-transformer to v1.126.2
a884539 
🔒 Scanned for secrets using gitleaks 8.30.0
@atzoum @achettyiitr
fixup! chore: bump rudder-transformer to v1.126.2
31be722 
🔒 Scanned for secrets using gitleaks 8.30.0
@achettyiitr

achettyiitr commented Mar 18, 2026

Copy link
Copy Markdown
Member Author

Cherry picked commits from #6774 so that the tests can pass.

@codecov

codecov Bot commented Mar 18, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 22.22222% with 7 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release/1.70.x@599307d). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...yncdestinationmanager/snowpipestreaming/channel.go 0.00% 5 Missing ⚠️
...tionmanager/snowpipestreaming/snowpipestreaming.go 33.33% 1 Missing and 1 partial ⚠️
Additional details and impacted files 
@@                Coverage Diff                @@
##             release/1.70.x    #6777   +/-   ##
=================================================
  Coverage                  ?   79.73%           
=================================================
  Files                     ?      565           
  Lines                     ?    63238           
  Branches                  ?        0           
=================================================
  Hits                      ?    50421           
  Misses                    ?     9838           
  Partials                  ?     2979

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

krishna2020
krishna2020 reviewed Mar 18, 2026
View reviewed changes
Comment thread go.mod
github.com/rudderlabs/rudder-observability-kit v0.0.6
github.com/rudderlabs/rudder-schemas v0.10.0
github.com/rudderlabs/rudder-transformer/go v1.122.0
github.com/rudderlabs/rudder-transformer/go v1.126.2-beta

@krishna2020 krishna2020 Mar 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ItsSudip Is the github action not yet fixed ?

@achettyiitr achettyiitr changed the base branch from master to release/1.70.x March 18, 2026 09:46
shekhar-rudder
shekhar-rudder reviewed Mar 18, 2026
View reviewed changes
Comment thread router/batchrouter/asyncdestinationmanager/snowpipestreaming/snowpipestreaming.go Outdated
@achettyiitr
chore: review comments
727b9a9 
🔒 Scanned for secrets using gitleaks 8.29.1
@achettyiitr
chore: review comments
a4f03e5 
🔒 Scanned for secrets using gitleaks 8.29.1
@achettyiitr achettyiitr force-pushed the fix.snowpipe-streaming-remove-authz branch from b9e04fa to a4f03e5 Compare March 18, 2026 15:13
@achettyiitr
chore: test fix
e09b8c4 
🔒 Scanned for secrets using gitleaks 8.29.1
@achettyiitr achettyiitr merged commit 3ecd5cd into release/1.70.x Mar 19, 2026
70 checks passed
@achettyiitr achettyiitr deleted the fix.snowpipe-streaming-remove-authz branch March 19, 2026 07:04
@rudderstack-github-actions rudderstack-github-actions Bot mentioned this pull request Mar 19, 2026
Merged
achettyiitr pushed a commit that referenced this pull request Mar 19, 2026
@rudderstack-github-actions
chore: release 1.70.1 (#6783)
9cdce95 
🤖 I have created a release *beep* *boop*
---


##
[1.70.1](v1.70.0...v1.70.1)
(2026-03-19)


### Bug Fixes

* snowpipe streaming remove authz error handling
([#6777](#6777))
([3ecd5cd](3ecd5cd))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
@rudderstack-github-actions rudderstack-github-actions Bot mentioned this pull request Mar 19, 2026
Merged
atzoum added a commit that referenced this pull request Mar 19, 2026
@rudderstack-github-actions @achettyiitr @atzoum
chore: sync release v1.70.1 to main branch (#6784)
5eaffb6 
# Description

Syncing patch release v1.70.1 to main branch

**↓↓ Please review and edit commit overrides before merging ↓↓**

BEGIN_COMMIT_OVERRIDE
fix: snowpipe streaming remove authz error handling (#6777)
END_COMMIT_OVERRIDE

---------

Co-authored-by: Akash Chetty <achetty.iitr@gmail.com>
Co-authored-by: Aris Tzoumas <atzoum@gmail.com>
Co-authored-by: Aris Tzoumas <atzoumas@rudderstack.com>
Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
This was referenced Mar 23, 2026
Closed
Closed
Closed
Merged
itsmihir pushed a commit that referenced this pull request Mar 23, 2026
@rudderstack-github-actions
chore: prerelease 1.71.0-rc.1 (#6793)
ecf41e2 
🤖 I have created a release *beep* *boop*
---


##
[1.71.0-rc.1](v1.70.0...v1.71.0-rc.1)
(2026-03-23)


### Bug Fixes

* snowpipe streaming remove authz error handling
([#6777](#6777))
([5eaffb6](5eaffb6))


### Miscellaneous

* bump rudder-schemas to 0.11.0
([#6779](#6779))
([7c7296b](7c7296b))
* bump rudder-transformer to v1.126.2
([#6774](#6774))
([69692ac](69692ac))
* force release 1.71.0
([#6791](#6791))
([9cf21c7](9cf21c7))
* **jobsdb:** partition migration improvements
([#6772](#6772))
([57b1c70](57b1c70))
* sync release v1.70.0 to main branch
([#6775](#6775))
([74125a6](74125a6))
* ut mirror filtering
([#6776](#6776))
([fa4d1c4](fa4d1c4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
@rudderstack-github-actions rudderstack-github-actions Bot mentioned this pull request Mar 23, 2026
Merged
atzoum added a commit that referenced this pull request Mar 23, 2026
@rudderstack-github-actions @achettyiitr @atzoum
chore: sync release v1.70.2 to main branch (#6797)
910f421 
# Description

Syncing patch release v1.70.2 to main branch

**↓↓ Please review and edit commit overrides before merging ↓↓**

BEGIN_COMMIT_OVERRIDE
fix: snowpipe streaming remove authz error handling (#6777)
fix: router panics when invalid header format is returned by transformer
(#6795)
END_COMMIT_OVERRIDE

---------

Co-authored-by: Akash Chetty <achetty.iitr@gmail.com>
Co-authored-by: Aris Tzoumas <atzoum@gmail.com>
Co-authored-by: Aris Tzoumas <atzoumas@rudderstack.com>
Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
This was referenced Mar 23, 2026
Merged
Closed
Merged
itsmihir pushed a commit that referenced this pull request Mar 23, 2026
@rudderstack-github-actions
chore: prerelease 1.71.0-rc.2 (#6800)
16108ab 
🤖 I have created a release *beep* *boop*
---


##
[1.71.0-rc.2](v1.70.0...v1.71.0-rc.2)
(2026-03-23)


### Bug Fixes

* router panics when invalid header format is returned by transformer
([#6795](#6795))
([910f421](910f421))
* snowpipe streaming remove authz error handling
([#6777](#6777))
([910f421](910f421))
* snowpipe streaming remove authz error handling
([#6777](#6777))
([5eaffb6](5eaffb6))


### Miscellaneous

* bump rudder-schemas to 0.11.0
([#6779](#6779))
([7c7296b](7c7296b))
* bump rudder-transformer to v1.126.2
([#6774](#6774))
([69692ac](69692ac))
* force release 1.71.0
([#6791](#6791))
([9cf21c7](9cf21c7))
* **jobsdb:** partition migration improvements
([#6772](#6772))
([57b1c70](57b1c70))
* sync release v1.70.0 to main branch
([#6775](#6775))
([74125a6](74125a6))
* ut mirror filtering
([#6776](#6776))
([fa4d1c4](fa4d1c4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
itsmihir pushed a commit that referenced this pull request Mar 23, 2026
@rudderstack-github-actions
chore: release 1.71.0 (#6798)
6e2a824 
🤖 I have created a release *beep* *boop*
---


##
[1.71.0](v1.70.0...v1.71.0)
(2026-03-23)


### Bug Fixes

* router panics when invalid header format is returned by transformer
([#6795](#6795))
([910f421](910f421))
* snowpipe streaming remove authz error handling
([#6777](#6777))
([910f421](910f421))
* snowpipe streaming remove authz error handling
([#6777](#6777))
([5eaffb6](5eaffb6))


### Miscellaneous

* bump rudder-schemas to 0.11.0
([#6779](#6779))
([7c7296b](7c7296b))
* bump rudder-transformer to v1.126.2
([#6774](#6774))
([69692ac](69692ac))
* force release 1.71.0
([#6791](#6791))
([9cf21c7](9cf21c7))
* **jobsdb:** partition migration improvements
([#6772](#6772))
([57b1c70](57b1c70))
* sync release v1.70.0 to main branch
([#6775](#6775))
([74125a6](74125a6))
* ut mirror filtering
([#6776](#6776))
([fa4d1c4](fa4d1c4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@krishna2020 krishna2020 krishna2020 approved these changes

@shekhar-rudder shekhar-rudder shekhar-rudder approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@achettyiitr @krishna2020 @shekhar-rudder @atzoum