Skip to content
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

Update Node to 6.13.0 LTS + current lib versions #8

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

dbtlr
Copy link

@dbtlr dbtlr commented Jan 11, 2018

This PR does the following:

  • Updates recommendation for Node to the 6.x LTS: 6.13.0
  • Updates all dependencies to their newest versions
  • Swaps Jade for Pug, which is the newer library
  • Basic code updates to update syntax for Express v4
  • Adds Express libraries that were split off from core that now must be included explicitly
  • Removes the /stats page
  • Removes basic auth (as that was basically protecting the stats page)
  • Removes Socket.io functionality, as we don't need real-time stats anymore

How to test:

  • Install and use Node 6.13.0 (nvm makes this easy)
  • Run npm install
  • Run npm start
  • Visit http://localhost:3000 -> you should see a simple HTML page
  • Run curl -X POST -d "url=https://www.learnboost.com" localhost:3000 from another terminal
  • This will return a url like: https://lrn.cc/0 You should be able to visit http://localhost:3000/0 to verify your redirect worked.

@dbtlr dbtlr changed the title Update Node to 8.9.4 + current lib versions Update Node to 6.13.0 LTS + current lib versions Feb 22, 2018
@dbtlr
Copy link
Author

dbtlr commented Feb 22, 2018

I updated this to use 6.13.0 and it works perfectly.

I updated the title and description to reflect this.

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.

1 participant