Skip to content
This repository has been archived by the owner on Apr 19, 2023. It is now read-only.

Update dependencies, upgrade to Express 4 #21

Closed
wants to merge 7 commits into from

Conversation

ericelliott
Copy link
Contributor

This PR updates all dependencies to the latest versions and upgrades to Express 4.

It also DROPS SUPPORT for Node <0.10.x.

There are a number of security issues with the Express 2.x implementation, including vulnerability to the bodyParser() multipart DoS exploit.

@ericelliott
Copy link
Contributor Author

Build failure on .6 and .8 is expected. Update the Travis config to account for the dropped support.

<div class="col-md-6">
<%-partial ('partial/display_logs_list', {logs:warning_events, log_type:'Warning'})%>
</div>
<%- include partial/display_logs_list.html %>
Copy link
Owner

Choose a reason for hiding this comment

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

Why does this needed to be changed?
We added some duplication in display_log_list.html with it.

@iloire
Copy link
Owner

iloire commented Mar 22, 2015

I have merged this changes in to master and bumped to 2.0.0 (since it breaks compatibility)
Thanks!

@iloire iloire closed this Mar 22, 2015
@ericelliott
Copy link
Contributor Author

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants