Skip to content

[sync] fix(security): move window.location.origin check inside isValidRedirectPath to fix SSR crash (#1527)#2830

Closed
tea-artist wants to merge 1 commit intodevelopfrom
sync/ee-20260325-023645
Closed

[sync] fix(security): move window.location.origin check inside isValidRedirectPath to fix SSR crash (#1527)#2830
tea-artist wants to merge 1 commit intodevelopfrom
sync/ee-20260325-023645

Conversation

@tea-artist
Copy link
Contributor

🔄 Automated sync from EE repository.

6 commit(s) synced since last sync.

Authors

Included commits

Latest source commit: teableio/teable-ee@ff75939

This PR was automatically created by the sync workflow.

…dRedirectPath to fix SSR crash (#1527)

Synced from teableio/teable-ee@ff75939

Co-authored-by: Aries X <caoxing9@gmail.com>
Co-authored-by: Bieber <artist@teable.io>
Co-authored-by: nichenqin <nichenqin@hotmail.com>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 23522213237

Details

  • 23 of 47 (48.94%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall first build on sync/ee-20260325-023645 at 63.122%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/nestjs-backend/src/event-emitter/event-job/fallback/fallback-queue.service.ts 5 7 71.43%
apps/nestjs-backend/src/event-emitter/event-job/fallback/local-queue.provider.ts 13 15 86.67%
apps/nestjs-backend/src/db-provider/select-query/postgres/select-query.postgres.spec.ts 0 5 0.0%
apps/nestjs-backend/src/features/auth/social/controller.adapter.ts 0 6 0.0%
apps/nestjs-backend/src/features/record/open-api/record-open-api-v2.service.ts 5 14 35.71%
Totals Coverage Status
Change from base Build 23486990488: 63.1%
Covered Lines: 25254
Relevant Lines: 40008

💛 - Coveralls

@tea-artist tea-artist closed this Mar 25, 2026
@tea-artist tea-artist deleted the sync/ee-20260325-023645 branch March 25, 2026 03:34
@github-actions
Copy link
Contributor

🧹 Preview Environment Cleanup

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.

3 participants