Christopher DeCairos

Limit actually limits to one less

4

https://github.com/ded/express-limiter/blob/99c10ed2de886dda2589b5af2bc06f445caf2ad6/index.js#L37 You will always be limited to one less than the limit you set. For example, if I set limit to `1` it will decrement `remaining`, save the JSON to...

"level" configuration appears to be ignored

3

[According to the pa11y documentation](https://github.com/pa11y/pa11y#command-line-interface) you should be able to pass `level` in your configuration file to support a few different use-cases: - `error`: exit with a code of 2...

Update example server

1

The example server needs to be updated to use Hapi 17

I cloned the repo, installed the deps, and ran the server, and found that a single test passes in the browser. Running `npm test` from the command line shows no...

Commented lines loaded by habitat.load

2

I'm seeing keys on the object returned by `env.all()` like: ``` js { '# export DB_HOST': '', '# export DB_USERNAME': 'root', '# export DB_PASSWORD': '', '# export DB_DATABASE': 'local_webmakers', '#...

I forked the original repo a while back, and added in far more configuration options, including the ability to deploy team and enterprise editions, along with some docs. https://github.com/cadecairos/mattermost-heroku/ This...

Use PORT to specify port value

4

Fixes #9

[A recent commit](https://github.com/mattermost/mattermost-integration-giphy/commit/0bf28ea4e1f5fc06dcee02bd2e86ca09d0a832e6#diff-d355af669766ee3562643e8e5034f715L96) changed the name of the environment variable that specifies the port number to bind to. Heroku requires that this variable be named `PORT`, otherwise the app will...

https://github.com/mozilla/advocacy.mozilla.org/blob/e1dcf4e169f645b03fff9ed8e98ae18f4dbdc80c/scripts/csp-directives.js#L37 replace with `https://comments.mozillafoundation.org` https://bugzilla.mozilla.org/show_bug.cgi?id=1617390