Skip to content

Fix event handler account scoping - #1911

Merged
zacchua-stripe merged 4 commits into
masterfrom
zacchua/fix-event-handler-account-context
Sep 17, 2026
Merged

zacchua-stripe merged 4 commits into
masterfrom
zacchua/fix-event-handler-account-context

Conversation

@zacchua-stripe

@zacchua-stripe zacchua-stripe commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Why?

Follow up to stripe/stripe-node#2850 (comment). Event notification callback clients inherited the original client’s Stripe-Account configuration while also applying the event’s Stripe-Context, which is an error. Hence we drop stripe-account

What?

  • Clear the Stripe account when creating an event-context callback client.
  • Add regression coverage verifying that callback clients use the event’s Stripe context without a Stripe account.

See Also

http://go/j/RUN_DEVSDK-3063

Configuration

  • skip-changefile: This PR is not a user-facing change, so there's no changefile.

zacchua-stripe and others added 2 commits September 16, 2026 11:54
Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>
Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>
@zacchua-stripe
zacchua-stripe marked this pull request as ready for review September 16, 2026 19:39
@zacchua-stripe
zacchua-stripe requested a review from a team as a code owner September 16, 2026 19:39
@zacchua-stripe
zacchua-stripe requested review from kidus-stripe and removed request for a team September 16, 2026 19:39
@zacchua-stripe
zacchua-stripe enabled auto-merge (squash) September 17, 2026 00:13
@zacchua-stripe
zacchua-stripe merged commit 329c702 into master Sep 17, 2026
20 checks passed
@zacchua-stripe
zacchua-stripe deleted the zacchua/fix-event-handler-account-context branch September 17, 2026 18:30
zacchua-stripe added a commit that referenced this pull request Sep 18, 2026
* rename changefiles that don't adhere to new naming rules

* Support `EventNotification`s with singleton related objects (#1904)

* support related singleton objects

* add missing changefile

* disabled reportUnnecessaryTypeIgnoreComment (#1912)

* Add option to suppress Stripe notices (#1909)

* Add option to suppress Stripe notices

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add changelog entry for notice suppression

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Clarify Stripe notice suppression instructions

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: codex <noreply@openai.com>

* Fix event handler account scoping (#1911)

* Fix event handler account scoping

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add PR URL to changefile

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: codex <noreply@openai.com>

* Remove deprecated StripeObject.request method

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add changelog entry for StripeObject.request removal

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Remove changelog section metadata

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: David Brownman <xavdid@stripe.com>
Co-authored-by: David Brownman <1231935+xavdid@users.noreply.github.com>
Co-authored-by: jar-stripe <jar@stripe.com>
Co-authored-by: codex <noreply@openai.com>
jar-stripe added a commit that referenced this pull request Sep 23, 2026
* Add option to suppress Stripe notices (#1909)

* Add option to suppress Stripe notices

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add changelog entry for notice suppression

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Clarify Stripe notice suppression instructions

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: codex <noreply@openai.com>

* Fix event handler account scoping (#1911)

* Fix event handler account scoping

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add PR URL to changefile

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: codex <noreply@openai.com>

* Document Stripe notice suppression (#1923)

Committed-By-Agent: goose
Orbit-Session-Id: d60d503e-7c8a-4aa0-89d1-37cc435afc02

---------

Co-authored-by: zacchua-stripe <zacchua@stripe.com>
Co-authored-by: codex <noreply@openai.com>
Co-authored-by: jar-stripe <jar@stripe.com>
Co-authored-by: Stripe OpenAPI <105521251+stripe-openapi[bot]@users.noreply.github.com>
jar-stripe added a commit that referenced this pull request Sep 29, 2026
* rename changefiles that don't adhere to new naming rules

* Support `EventNotification`s with singleton related objects (#1904)

* support related singleton objects

* add missing changefile

* disabled reportUnnecessaryTypeIgnoreComment (#1912)

* Add option to suppress Stripe notices (#1909)

* Add option to suppress Stripe notices

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add changelog entry for notice suppression

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Clarify Stripe notice suppression instructions

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: codex <noreply@openai.com>

* Fix event handler account scoping (#1911)

* Fix event handler account scoping

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add PR URL to changefile

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: codex <noreply@openai.com>

* Remove deprecated StripeObject.request method

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Add changelog entry for StripeObject.request removal

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

* Remove changelog section metadata

Committed-By-Agent: codex
Co-authored-by: codex <noreply@openai.com>

---------

Co-authored-by: David Brownman <xavdid@stripe.com>
Co-authored-by: David Brownman <1231935+xavdid@users.noreply.github.com>
Co-authored-by: jar-stripe <jar@stripe.com>
Co-authored-by: codex <noreply@openai.com>
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.

2 participants