FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore: add a flag to disable local login while setting up by siddharthsarda · Pull Request #11091 · NodeBB/NodeBB · GitHub

/ NodeBB Public

chore: add a flag to disable local login while setting up - #11091

Open
siddharthsarda wants to merge 1 commit into
NodeBB:masterfrom
siddharthsarda:feat/add_config_flag_to_disable_local_login
Open

chore: add a flag to disable local login while setting up#11091
siddharthsarda wants to merge 1 commit into
NodeBB:masterfrom
siddharthsarda:feat/add_config_flag_to_disable_local_login

Conversation

siddharthsarda commented Dec 16, 2022
edited
Loading

Copy link
Copy Markdown

The issue is described here: https://community.nodebb.org/post/90464. I want to be able to control local login when deploying NodeBB on a managed container service like AWS Apprunner. ./nodebb setup always grants local login privileges even if the database has them disabled (probably because they are booleans with the default as false, not ideal for most cases)

CLAassistant commented Dec 16, 2022
edited
Loading

Copy link
Copy Markdown


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.


Siddharth Sarda seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL