Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: update got to use built-in caching #5090

Merged
merged 3 commits into from
Jan 4, 2023
Merged

chore: update got to use built-in caching #5090

merged 3 commits into from
Jan 4, 2023

Conversation

AlCalzone
Copy link
Member

@AlCalzone AlCalzone commented Sep 20, 2022

This PR removes our caching band-aid for the firmware update service in favor of the now-fixed built-in caching in got. In addition, it makes the service URL configurable through the env variable ZWAVEJS_FW_SERVICE_URL.

fixes: #5213

@AlCalzone AlCalzone marked this pull request as ready for review January 4, 2023 10:51
@AlCalzone
Copy link
Member Author

@zwave-js-bot approve

@AlCalzone AlCalzone enabled auto-merge (squash) January 4, 2023 10:55
import os from "os";
import path from "path";
import "reflect-metadata";
import { Driver } from "zwave-js";

const wait = _wait;

Check notice

Code scanning / CodeQL

Unused variable, import, function or class

Unused variable wait.
@AlCalzone AlCalzone merged commit f0a6d93 into master Jan 4, 2023
@AlCalzone AlCalzone deleted the update-got branch January 4, 2023 11:00
AlCalzone added a commit that referenced this pull request Jan 16, 2023
### Features
* Support OTW firmware updates of 500 and 700+ series controllers (#5321, #5326)
* Parallel firmware updates and soft/hard reset during ongoing firmware updates is now prevented (#5220)
* Added a readonly `rfRegion` property to the `Controller` class (#5288)
* Added support for requesting region-specific firmware updates from the update service (#5296)
* Allow configuring the number of kept logfiles (#5294)
* Added support for the `Inclusion Controller CC` which allows secondary controllers in the network to include devices on behalf of Z-Wave JS (#4851)
* Added support for scanning QR codes that only contain the DSK, as well as pre-filling the DSK before the inclusion process (#5309)

### Bugfixes
* Known Wake Up CC version no longer gets overwritten with 1 (#5261)
* Surrounding whitespace in S2 or SmartStart QR codes now gets ignored (#5295)
* Always use S2 for endpoint communication if the node uses S2 (#5310)
* Distinguish between protocol and SDK version on 500 series (#5323)

### Config file changes
* Add metadata for Sensative strips (#5223)
* Add Ring Retrofit Alarm Kit (#5299)
* Correct value size for some Nortek/GoControl device params (#5297)
* Correct low temp threshold for Aeotec aërQ (#5286)
* Add fingerprint for Fibaro FGWP102 (#5280)
* Add metadata to Aeotec aërQ (#5224)
* Allow higher minimum dim level for Inovelli LZW31-SN, FW 1.57+ (#5181)
* Add Zooz 800 series controllers (#5324)

### Changes under the hood
* Removed a workaround for broken caching in the `got` library (#5090)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make URL for firmware update service an environment variable
2 participants