Gzip / Caching Improvements #16

brianhanifin · 2014-06-21T00:57:53Z

First, I modified nodeStatic.Server(".") to include a 30 day cache and enabled gzip compression. It may only make a small difference with the .mp3 files, but the js and css files are much smaller when gzipped. I added gzipped versions of the MP3 files (e.g. alert.mp3.gz).

var staticServer = new nodeStatic.Server(".", { cache:2592000, gzip:true });

Next, using appcache-node I implemented an "offline" HTML 5 Application Cache. Below is the generated nightscout.appcache file. The "NETWORK:" group defines the files that should never be cached offline. By specifying an asterisk we are saying that no other file should be cached (socket.io won't work if cached for example).

nightscout.appcache

CACHE MANIFEST

CACHE:
http://localhost:1337/audio/alarm.mp3
http://localhost:1337/audio/alarm2.mp3
http://localhost:1337/audio/alarm.mp3.gz
http://localhost:1337/audio/alarm2.mp3.gz
http://localhost:1337/css/dropdown.css
http://localhost:1337/css/main.css
http://localhost:1337/js/client.js
http://localhost:1337/js/dropdown.js
http://localhost:1337/favicon.ico

NETWORK:
*

# Fri Jun 20 2014 17:49:43 GMT-0700 (PDT)

I believe this is proof that the .mp3 files are being cached (showing a 304 Not Modified status). This as screenshot of Live HTTP Headers:

This is my first big stab at contributing. What do you think? I have it working both locally and on a test Azure server.

importing lastest nightscout master code

Trying some experiments to see if I can serve alarm.mp3.gz files instead of the uncompressed originals.

The .mp3 files will not be sent compressed without this.

changed gzip:'true’… According to the node-static read me this shouldn’t work, but it works to compress all files. /audio/alarm.mp3 drops from 587,716 bytes to 4,822! TEST OUTPUT macbookpro:audio dev$ curl http://localhost:1337/audio/alarm.mp3 --silent --write-out "%{size_download}\n" --output /dev/null 587716 curl --compress http://localhost:1337/audio/alarm.mp3 --silent --write-out "%{size_download}\n" --output /dev/null 4822

Added additional instructions to get my.env working.

Used appcache-node to generate a nightscout.appcache file which includes directives on which files to cache.

(socket.io and bower)

bewest · 2014-06-21T01:40:07Z

👍 fantastic, thank you!

bewest · 2014-06-21T01:46:08Z

You might want to change your mongo db password, if you have not already.

This should help stay under Azure's free quotas.

Gzip / Caching Improvements

brianhanifin · 2014-06-21T01:51:01Z

I created a second site on the same Azure account today. Will that cause me a problem?

Here is further proof of the improvement this made. :)

rnpenguin · 2014-06-21T01:56:18Z

Nice.

@bewest should we create the following for nightscout specific:

brianhanifin · 2014-06-21T02:05:08Z

Oops. I didn't intend to leave those in there. I saw those on another project and thought they were interesting services, so I was experimenting with them.

This provides a project chatroom. I was wondering if that would be useful for us to discuss issues outside of the issue tracker and Facebook group.

I'm not sure if I am taking full advantage of this, but it is supposed to warn you of errors.

This service lets you know if your dependancies have newer versions.

bewest · 2014-06-21T02:08:17Z

Yes. These look good.
Travis isn't doing anything ATM, but it would not be hard for someone to
add a test and is worth keeping.

On Fri, Jun 20, 2014 at 6:56 PM, rnpenguin [email protected] wrote:

Nice.

@bewest https://github.com/bewest should we create the following for
nightscout specific:

[image: Build Status]
https://travis-ci.org/brianhanifin/cgm-remote-monitor
[image: Gitter chat] https://gitter.im/brianhanifin/cgm-remote-monitor
[image: Dependency Status]
https://david-dm.org/brianhanifin/cgm-remote-monitor

—
Reply to this email directly or view it on GitHub
#16 (comment)
.

rnpenguin · 2014-06-21T02:13:07Z

Agree. Should we set it up for nightscout specifically though

bewest · 2014-06-21T02:17:51Z

I turned on travis just now.

bewest · 2014-06-21T02:23:15Z

https://gitter.im/nightscout

https://david-dm.org/nightscout/cgm-remote-monitor - I'm not crazy about this one, we pinned down socket.io on purpose. It'd be pretty cool if instead, they measured whether or not the dependency chain is precise and can be resolved rather than how old they are.

bewest · 2014-06-21T02:27:18Z

rnpenguin · 2014-06-21T02:28:07Z

:).

That would be better, I need to look into it a bit more.

brianhanifin · 2014-06-21T03:32:26Z

You can't have a chatroom at your username, but you can create a custom chat room named dev for example. Oh look, you can shorten the title on the badge, and still link to a longer chat room name.

[![Gitter chat](https://badges.gitter.im/nightscout.png)](https://gitter.im/nightscout/dev)

brianhanifin · 2014-06-21T03:40:28Z

Oh, and I changed the password. Thanks for the reminder.

jwedding · 2014-06-22T21:53:50Z

Guys, I think I tried to pull in the wrong direction. Please ignore/cancel that request. I'll have to do this when I'm paying better attention.

robynec75 · 2014-06-28T23:18:32Z

I think I hit my limit because of an alarm running on an open window on the computer while we were out today. Can I get some help in adding this caching behavior to mine? Thanks!

brianhanifin · 2014-06-28T23:26:32Z

This version of the code didn't quite have the effect we were hoping for. I have a new branch that is about ready, and caches the mp3 files really well. You can merge this into your copy. Go to the link below, and create a pull request to your master branch (make sure your branch is on the left, and mine is on the right).

https://github.com/brianhanifin/cgm-remote-monitor/express-cache

robynec75 · 2014-06-29T01:56:48Z

How do I create a pull request? Also when I clicked on the link I get a 404 error. I just did my setup yesterday so this will be my first update of any kind and I don't really know how to do it. Thanks!

brianhanifin · 2014-06-29T02:06:36Z

robynec75 · 2014-06-29T02:47:21Z

Thanks! I'm pretty sure i did it right.

Dev sync

…mol-l Adds support for mmol/L units. fixes nightscout#16

Navid 2022 10 26

a

feat(ai-eval): Add 14-day limit to AI evaluation report

brianhanifin added 15 commits June 19, 2014 21:35

Merge pull request #1 from nightscout/master

e151650

importing lastest nightscout master code

Exploring caching and gzip techniques

d3c0f4a

Create database_configuration.json

0b158da

adding Content-Encoding: gzip to header

f9af718

Trying some experiments to see if I can serve alarm.mp3.gz files instead of the uncompressed originals.

changed gzip:/^/mp3/ to gzip:true

637ec91

Adding travis support

f034e7a

changed gzip:true back to gzip:/^/mp3/

1f00fa1

The .mp3 files will not be sent compressed without this.

test

5fc3a02

Update database_configuration.json

ec54ca5

Update README.md

a05b1c8

Added additional instructions to get my.env working.

gzip compression is working

8cb346a

Enabled HTML5 local storage cache

781722e

Used appcache-node to generate a nightscout.appcache file which includes directives on which files to cache.

Added appcache-node to package.json

5cf77ff

HTML5 app cache: adding more client libraries

c9a0e36

(socket.io and bower)

bewest added a commit that referenced this pull request Jun 21, 2014

Merge pull request #16 from brianhanifin/feature/cache

f46f3a3

Gzip / Caching Improvements

bewest merged commit f46f3a3 into nightscout:master Jun 21, 2014

brianhanifin deleted the feature/cache branch June 24, 2014 23:47

PieterGit pushed a commit that referenced this pull request Feb 2, 2019

Merge pull request #16 from nightscout/dev

dd91d05

Dev sync

mrspouse pushed a commit to mrspouse/cgm-remote-monitor that referenced this pull request Apr 3, 2022

Adds support for mmol/L units. fixes nightscout#16

7d7a868

mrspouse pushed a commit to mrspouse/cgm-remote-monitor that referenced this pull request Apr 3, 2022

Merge pull request nightscout#17 from timoschlueter/feature/support-m…

7dfa95c

…mol-l Adds support for mmol/L units. fixes nightscout#16

justinthomas pushed a commit to justinthomas/cgm-remote-monitor that referenced this pull request Nov 29, 2022

Merge pull request nightscout#16 from Navid200/Navid_2022_10_26

72077c4

Navid 2022 10 26

Ki-ches added a commit to Ki-ches/cgm-remote-monitor that referenced this pull request Jun 20, 2024

Merge pull request nightscout#16 from nightscout/master

6527dd9

a

psonnera pushed a commit to psonnera/cgm-remote-monitor that referenced this pull request Oct 18, 2025

Merge pull request nightscout#16 from xannasavin/feat/ai-report-plugin-1

806229f

feat(ai-eval): Add 14-day limit to AI evaluation report

Gzip / Caching Improvements #16

Gzip / Caching Improvements #16

Uh oh!

Conversation

brianhanifin commented Jun 21, 2014

nightscout.appcache

Uh oh!

bewest commented Jun 21, 2014

Uh oh!

bewest commented Jun 21, 2014

Uh oh!

brianhanifin commented Jun 21, 2014

Uh oh!

rnpenguin commented Jun 21, 2014

Uh oh!

brianhanifin commented Jun 21, 2014

Uh oh!

bewest commented Jun 21, 2014

Uh oh!

rnpenguin commented Jun 21, 2014

Uh oh!

bewest commented Jun 21, 2014

Uh oh!

bewest commented Jun 21, 2014

Uh oh!

bewest commented Jun 21, 2014

Uh oh!

rnpenguin commented Jun 21, 2014

Uh oh!

brianhanifin commented Jun 21, 2014

Uh oh!

brianhanifin commented Jun 21, 2014

Uh oh!

jwedding commented Jun 22, 2014

Uh oh!

robynec75 commented Jun 28, 2014

Uh oh!

brianhanifin commented Jun 28, 2014

Uh oh!

robynec75 commented Jun 29, 2014

Uh oh!

brianhanifin commented Jun 29, 2014

Uh oh!

robynec75 commented Jun 29, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants