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

nightscout update #5

Merged
merged 85 commits into from
Jul 27, 2022
Merged

nightscout update #5

merged 85 commits into from
Jul 27, 2022

Conversation

njmo
Copy link
Owner

@njmo njmo commented Jul 27, 2022

No description provided.

sulkaharo and others added 30 commits February 23, 2021 14:54
* Change packaging to include full moment.js locale library to fix date processing issues for some users
* Create docker-compose.yml file for easy deployment

* Restore invalid password, to force change
* New translations en.json (Hungarian)

* New translations en.json (Hungarian)

* New translations en.json (Czech)

* New translations en.json (Hungarian)

* New translations en.json (Hungarian)

* New translations en.json (Hungarian)

* New translations en.json (Hungarian)

* New translations en.json (Hungarian)

* New translations en.json (Hungarian)

* New translations en.json (Russian)

* New translations en.json (Romanian)

* New translations en.json (Italian)

* New translations en.json (Italian)

* New translations en.json (Tamil)

* New translations en.json (Portuguese)

* New translations en.json (Slovenian)

* New translations en.json (Slovenian)

* New translations en.json (Slovenian)

* New translations en.json (Slovenian)

* New translations en.json (Slovenian)

* New translations en.json (Slovenian)

* New translations en.json (Chinese Simplified)

* New translations en.json (Norwegian Bokmal)

* New translations en.json (French)

* New translations en.json (Turkish)

* New translations en.json (Turkish)

* New translations en.json (Chinese Simplified)

* New translations en.json (Chinese Simplified)
Use npm update --save share2nightscout-bridge to update package-lock.json and
package.json.  This should assist in updates intended to help unify Dexcom Account management.
…bridge-update

bump share2nightscout-bridge version
This change allows each route to express policies for interpreting and parsing
the request body.  Before this change uploads to entries or treatments api
would error if they were larger than 100Kb due to the preference being set for
the whole server.  This change removes the global preference in favor of
allowing each route to choose it's own request size limit.

We also refactor usage of body-parser to be more consistent throughout the code
base.  Most routes can use jsonParser, rawPraser, and urlencodedParser provided
by the common wares component.  Anything doing something else should be called
out as such.  For example, treatments, activity, and entries allow uploads up
to 50Mb.  Other v1 endpoints are using the common configuration set to 1Mb.
respect per route body-parser configuration
Many proxies (NGINX included) will generate a list of IPs in v4 and
v6 formats.  The forwarded-for library is a well-maintained library for
express that sanitizes, checks, and validates trusted proxy IPs.  This helps
eliminate XSS or other attempts to inject invalid material through the
x-forwarded-for header.

Co-authored-by: Sulka Haro <[email protected]>
sulkaharo and others added 29 commits November 19, 2021 12:22
Forcing a newer version of share2nightscout-bridge in order to keep up
with Dexcom changes.
Include enhancements on the minimed-connect-to-nightscout plugin to include
fetching data from a Careportal account.
Using a ratio of 18 mg/ml to 1 mmol/l causes NightScout to report different values compared Dexcom app/receivers.

A more accurate value is based on 0.0555 mmol/l to 1 mg/dl. The inverse is 18.018018018 mg/dl to 1 mmol/l

This puts the value closer to the Dexcom app/receiver. Affected values:

|mg/dl|  18|18.018018018|
|----:|---:|-----------:|
|  109| 6.1|         6.0|
|  118| 6.6|         6.5|
|  127| 7.1|         7.0|
|  136| 7.6|         7.5|
|  145| 8.1|         8.0|
|  154| 8.6|         8.5|
|  163| 9.1|         9.0|
|  172| 9.6|         9.5|
|  181|10.1|        10.0|
|  190|10.6|        10.5|
|  199|11.1|        11.0|
|  208|11.6|        11.5|
|  217|12.1|        12.0|
|  226|12.6|        12.5|
|  235|13.1|        13.0|
|  244|13.6|        13.5|
|  253|14.1|        14.0|
|  262|14.6|        14.5|
|  271|15.1|        15.0|
|  280|15.6|        15.5|
|  289|16.1|        16.0|
|  298|16.6|        16.5|
|  307|17.1|        17.0|
|  309|17.2|        17.1|
|  316|17.6|        17.5|
|  318|17.7|        17.6|
|  325|18.1|        18.0|
|  327|18.2|        18.1|
|  334|18.6|        18.5|
|  336|18.7|        18.6|
|  343|19.1|        19.0|
|  345|19.2|        19.1|
|  352|19.6|        19.5|
|  354|19.7|        19.6|
|  361|20.1|        20.0|
|  363|20.2|        20.1|
|  370|20.6|        20.5|
|  372|20.7|        20.6|
|  379|21.1|        21.0|
|  381|21.2|        21.1|
|  388|21.6|        21.5|
|  390|21.7|        21.6|
|  397|22.1|        22.0|
|  399|22.2|        22.1|
Using share2nightscout-bridge, the device property is defined as 'share2'

displayBg should honour the Dexcom HIGH and LOW values when the device is share2

This ensures that NightScout returns "LOW" for 39mg/dl and "HIGH" for 401mg/dl as is reported in the Dexcom app/receiver.
This patches uses a newer versin of minimed-connect-to-nightscout
that is designed to work under the same version of node as
Nightscout.  It should result in Nightscout gaining a capability
to use Medtronic Careportal accounts for cloud to cloud
synchronization.
The `npm audit fix` in the `Dockerfile` is currently
deleting vulnerable packages. Although we do want to have package
security, this should be done in a development process, not as part
of the Docker build.

This commit also reduces the Docker image size and improves security.
This should force the coverage job to return with a success error code.
Codacy changed support for legacy projects.  This change should not cause
builds to fail and should not influence ability to merge pull requests.  Until
further work with coverage with codacy, this change emits a warning of "NO
COVERAGE" while allowing the build to pass, rather than failing the build.
When collecting date from Dexcom via share2bridge-dexcom
before we would poll at regular intervals.

Dexcom transmitters / apps refresh every 5 minutes give/take the inaccuracy in the transmitters clock.

If we poll every 5 minutes, we will randomly introduce up to 5 minutes of lag before we refresh.

For example, if we poll 4:59s after dexcom refreshed we will face a 4:59s lag seeing the latest data arrive

This patch introduces a refresh adjustment. We look at the timestamp of the most recent data item and aim to refresh
5 minutes plus a buffer after that timestamp.

for example, 5:20s after the most recent data time. the additional buffer gives dexcom a chance to push the data through their
application/infrastructure (after the app sends the data, it appears in testing to be around 20s before it is available in the API)

This reduces the lag time to a reliable 20 seconds regardless of when nightscout was started and keeps it in track as it drifts forwards and back.
The default interval is increasd from 2.5 minutes to 2.6 minutes
This change incorporates the lag between the app uploading
and the data being available in share2nightscout-bridge

This change also accomodates the optimisation of the refresh interval
based on the last collected data timestamp
…excom_fetch_times

Optimize time between polling share2nightscout-bridge to reduce ingest lag
@njmo njmo merged commit 447be08 into njmo:master Jul 27, 2022
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.

8 participants