-
Notifications
You must be signed in to change notification settings - Fork 138
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
GET data (as carepartner) (EU) results in Error: Request failed with status code 400 #39
Comments
The URL is retrieved from medtronic (see carelink.js)
Could it be that the body that is send to medtronic needs other information? And is there a way I can find that information? For completness: the log information:
|
same here. since today 1600 CEST I have the same error. Haven't found any solution yet. :( iPhone Apps work fine. |
same thing here 😒 |
In the URL, I see a mention of v6 what probably stands for Version 6. Maybe, if someone still knows the url for the previous version, we can fetch the data through the old url? |
Great idea
A simple swap for "v5" gives results! |
🥳 this hotfix did it. I added a new line 248 to carelink.js updating the function getConnectData() as follows: ` async function getConnectData() {
With this change it works again for me! Thanks @sirKitKat for the simple yet genius idea! |
Nice, this is, however, a temporary fix. It will work until Medtronic deprecates version 5. Nevertheless, it is working right now. :) |
confirm V5 also working for me |
I'm interested in collecting credentials and helping to document/implement this. |
I have a pull request for this. #37 Working on a release, ran into a snag with npm, but it would be really helpful for people to test it more completely while that gets resolved. |
I released |
Hi Ben, I sent you a message on gitter |
Actually, it can be fixed without reverting back to v5. To do this, in "carelink.js", the "getConnectData"-function should be changed where we build the body for the case
should become
where patientId should be the patient's username that you want to monitor. So I guess we need to introduce a new environment variable that contains that information and edit the body. (Credits to @sedy89: NightscoutFoundation/xDrip#2823 (comment)) |
I Create a quick PR with the changes that would be required: #40 |
There patches implementing M2M functionality in nightscout-connect, dev: nightscout/nightscout-connect#4. I'm thinking of deprecating this module in favor of the unifed nightscout-connect. We could leave the "v5 fix" in here, and then target all new M2M featureset in the new module. Are there still users depending on the older feature set? When is the ble endpoint used to poll for data instead of the M2M endpoint? Is the whole world m2m enabled now? |
@bewest the PR is merged in master now, but the hotfix is still in place in carelink.js
The fix of PR #41 works only with the latest endpoint (v6) not with V5... Removing the aforementioned lines should fix this. |
Nothing changed on my side but I suddenly started getting this error (Around 16:00 CEST)
The script is fetching the data from the following URL:
https://clcloud.minimed.eu/connect/carepartner/v6/display/message
Anyone an idea how to resolve this issue?
(Country: Belgium)
The text was updated successfully, but these errors were encountered: