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

Re-orders steps 6 and 7. #3162

Merged
merged 1 commit into from Jan 19, 2017

Conversation

Projects
None yet
3 participants
@dmark
Copy link
Contributor

dmark commented Jan 6, 2017

I believe the current order of steps 6 and 7 are incorrect. At step 6, there is no config.json file. Creating one using step 6 as a template and then running node server.js results in the error:

Error: Error parsing your JSON configuration file.

I believe the steps should be reversed (this is what worked for me):

  1. Run node server.js and enter the ticket number / URL from the Auth0 dashboard. This will create the config.json file for you.

  2. Edit config.json, adding your LDAP connection and authentication details.

  3. Run node server.js again.

Re-orders steps 6 and 7.
I believe the current order of steps 6 and 7 are incorrect. At step 6, there is no config.json file. Creating one using step 6 as a template and then running `node server.js` results in the error:

    Error: Error parsing your JSON configuration file.

I believe the steps should be reversed (this is what worked for me):

6. Run `node server.js` and enter the ticket number / URL from the Auth0 dashboard. This will create the config.json file for you.

7. Edit config.json, adding your LDAP connection and authentication details.

8. Run `node server.js` again.

@ntotten ntotten temporarily deployed to auth0-docs-content-pr-3162 Jan 12, 2017 Inactive

@ntotten ntotten temporarily deployed to auth0-docs-content-pr-3162 Jan 18, 2017 Inactive

@ntotten ntotten temporarily deployed to auth0-docs-content-pr-3162 Jan 19, 2017 Inactive

@mpaktiti

This comment has been minimized.

Copy link
Contributor

mpaktiti commented Jan 19, 2017

Looks good, thanks for the contribution @dmark ! 👍

@mpaktiti mpaktiti merged commit bc08a73 into auth0:master Jan 19, 2017

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