Skip to content

Add a section in contributing guide describing owner user creation and reset #16006

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

iocanel
Copy link

@iocanel iocanel commented Jun 4, 2025

Summary

The pull request introduces a section in the CONTRIBUTING.md that explain the process of owner user creation and reset.

Related Linear tickets, Github issues, and Community forum posts

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)

@CLAassistant
Copy link

CLAassistant commented Jun 4, 2025

CLA assistant check
All committers have signed the CLA.

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.

cubic reviewed 1 file and found no issues. Review PR in cubic.dev.

CONTRIBUTING.md Outdated
If for any reason the credentials are lost before setting up password recovery, you can either remove the database file or run the following SQL command to reset the owner user:

```
update settings SET value='false' where key='userManagement.isInstanceOwnerSetUp';
Copy link
Member

Choose a reason for hiding this comment

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

We would recommend using the server command over directly editing the database: https://docs.n8n.io/hosting/cli-commands/#user-management

Copy link
Author

Choose a reason for hiding this comment

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

Replaced sql with ./packages/cli/bin/n8n user-management:reset

@@ -169,6 +169,21 @@ To start n8n with tunnel:
./packages/cli/bin/n8n start --tunnel
```

## First run

The first time you run n8n locally you will be prompted to create an owner user.
Copy link
Member

Choose a reason for hiding this comment

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

Is this information really needed? it is the same as normally running n8n so this shouldn't be a surprise.

Copy link
Author

Choose a reason for hiding this comment

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

It's definitely not a suprise during first run. It still adds some context for users that do need to reset the credentials (e.g. people that tried running n8n locally once or twice and return back to it a couple of weeks later). So, I'd say let's keep it but I am perfectly fine with removing it.

@n8n-assistant n8n-assistant bot added community Authored by a community member in linear Issue or PR has been created in Linear for internal review labels Jun 4, 2025
@Joffcom
Copy link
Member

Joffcom commented Jun 4, 2025

Hey @iocanel,

Thanks for the PR, We have created "GHC-2341" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

@Joffcom
Copy link
Member

Joffcom commented Jun 4, 2025

Hey @iocanel,

Thanks for the PR, We have created "GHC-2342" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

@iocanel iocanel force-pushed the contributing-improvements branch from 2de44e0 to 071b346 Compare June 4, 2025 09:07
@iocanel iocanel requested a review from Joffcom June 6, 2025 06:20
@Joffcom
Copy link
Member

Joffcom commented Jun 6, 2025

@iocanel I will run this past the docs team and see what they think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member in linear Issue or PR has been created in Linear for internal review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants