Skip to content

Conversation

afitzek
Copy link
Contributor

@afitzek afitzek commented Oct 13, 2025

Summary

Introduces a new setting N8N_DISABLE_INVITE_LINK_EXPOSURE, which prevents the exposure of invite links to accounts that are allowed to create new users. This is a security hardening setting. Once SMTP settings are set up, this can be activated so that each new user receives their invite link only via email.

Related Linear tickets, Github issues, and Community forum posts

closes https://linear.app/n8n/issue/PAY-3966/zero-click-account-takeover

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Oct 13, 2025
@afitzek afitzek marked this pull request as ready for review October 13, 2025 21:01
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

2 issues found across 3 files

Prompt for AI agents (all 2 issues)

Understand the root cause of the following 2 issues and fix them.


<file name="packages/cli/src/controllers/__tests__/users.controller.test.ts">

<violation number="1" location="packages/cli/src/controllers/__tests__/users.controller.test.ts:112">
Update the test to set `disableInviteLinkExposure` so it matches the controller logic and the intended configuration.</violation>

<violation number="2" location="packages/cli/src/controllers/__tests__/users.controller.test.ts:231">
Switch this assignment to `disableInviteLinkExposure` so the test actually toggles the flag consumed by the controller.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

@afitzek afitzek force-pushed the pay-3966-zero-click-account-takeover branch from 4f1be8e to badeb98 Compare October 13, 2025 21:19
Copy link

codecov bot commented Oct 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant