update my site #6
Merged
Conversation
This file contains 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
I added a space to test Codeship for the tutorial.
…ies and /entries/current
…exception handling
Feature/base api
Wrap protected functions with: if (verifyAuthorization(req, res)) {}
Added HTTP Header Authorization
Auth: Added undefined validation
…ection to the api
…onitor into feature/base-api
aka moving /settings to bottom
…onitor into feature/base-api
This sets up two additional API endpoints explicitly used for testing. Also, constrain all parsing of environment variables to a single location. This change also creates no API at all unless there is an API secret defined. Also the API_SECRET is really just a shared key private key, we actually use the sha1 sum digest of the key.
to comply with recent change to Express. This gets rid of the depreciated warnings at the command line when launching the server.
At startup server.js now attempts to create the necessary collections. If they exist already they will NOT be overwritten. If default data population is necessary, a JSON object may be passed for population. Unless I am missing something, this means the users will NOT have to manually add the ‘settings’ collection. It will happen automatically in the background. I also set it up to initialize the Entries collection as ‘entries’ if no name has been provided, and the database does not exist. No default data population occurs for entries. This requires that the website be setup with a mongo user with write permissions. Of course we were going to need that for the API anyway.
per the guidance of http://package-json-validator.com/ and https://www.npmjs.org/doc/package.json.html
unless running on localhost. *I need to test this on Heroku and Azure.*
* Querystring values now replace underscores with a space, pluses are now treated as any other character. * Added type parameter to showNotifications(note, type). Now blue, green, red, and yellow background can be used (by passing the type as "info", "success", "urgent", "warn").
* Changed other links to text links.
@jasoncalabrese pointed out that some agents do odd things with underscores and that using a hyphen is more common. This also adds tests to ensure that the API_SECRET handling works as expected.
Cherry-picked in the wrong order This reverts commit 8e56ef1.
That way the CSS and HTML can stay enabled. When we are ready to add code to ui-utils.js we can just start running.
…nitor into rebase/settings
Adding warning + urgent colored backgrounds when BG is out of date by 10 or 20 minutes.
For parents monitoring multiple kids. Replaces "Nightscout" in the toolbar.
Add Settings
Base API - settings, basic auth, and experimental endpoints
wagnerpe
pushed a commit
that referenced
this pull request
Jun 22, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.