@autofleet/settings 1.1.1 → 1.1.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/index.js +2 -2
- package/dist/index.test.js +1 -1
- package/package.json +2 -2
- package/src/index.test.ts +1 -1
- package/src/index.ts +2 -2
package/dist/index.js
CHANGED
|
@@ -78,14 +78,14 @@ class SettingsManager {
|
|
|
78
78
|
this.settingsCache.set(cacheKey, waitingToNetwork, this.ttl);
|
|
79
79
|
const networkResponse = yield this.network.get(`/api/v1/settings/get-setting/${key}`, {
|
|
80
80
|
params: {
|
|
81
|
-
labels,
|
|
81
|
+
labels: labels.map((label) => JSON.stringify(label)),
|
|
82
82
|
},
|
|
83
83
|
timeout,
|
|
84
84
|
});
|
|
85
85
|
networkValue = networkResponse.data.value;
|
|
86
86
|
}
|
|
87
87
|
catch (error) {
|
|
88
|
-
logger.error('Cant get setting from network');
|
|
88
|
+
logger.error('Cant get setting from network', { error });
|
|
89
89
|
if (rejectOnFail) {
|
|
90
90
|
throw new CannotGetNetworkValueError();
|
|
91
91
|
}
|
package/dist/index.test.js
CHANGED
|
@@ -20,7 +20,7 @@ const serviceUrl = 'localhost:8085';
|
|
|
20
20
|
const mockSetting = (key, value, labels = undefined, response = 200) => {
|
|
21
21
|
const n = nock(`http://${serviceUrl}`)
|
|
22
22
|
.get(`/api/v1/settings/get-setting/${key}`)
|
|
23
|
-
.query(labels ? { labels } : undefined);
|
|
23
|
+
.query(labels ? { labels: labels.map((label) => JSON.stringify(label)) } : undefined);
|
|
24
24
|
return n.reply(response, { value });
|
|
25
25
|
};
|
|
26
26
|
const mockMultipleSettings = (keys, values, response = 200) => {
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@autofleet/settings",
|
|
3
|
-
"version": "1.1.
|
|
3
|
+
"version": "1.1.3",
|
|
4
4
|
"description": "",
|
|
5
5
|
"main": "dist/index.js",
|
|
6
6
|
"scripts": {
|
|
@@ -15,7 +15,7 @@
|
|
|
15
15
|
},
|
|
16
16
|
"dependencies": {
|
|
17
17
|
"@autofleet/logger": "^1.1.0",
|
|
18
|
-
"@autofleet/network": "^1.3.
|
|
18
|
+
"@autofleet/network": "^1.3.7",
|
|
19
19
|
"@types/jest": "^22.0.0",
|
|
20
20
|
"bluebird": "^3.7.2",
|
|
21
21
|
"dotenv": "^8.2.0",
|
package/src/index.test.ts
CHANGED
|
@@ -10,7 +10,7 @@ const serviceUrl = 'localhost:8085';
|
|
|
10
10
|
const mockSetting = (key: string, value: string | boolean | number | never, labels: any = undefined, response = 200) => {
|
|
11
11
|
const n = nock(`http://${serviceUrl}`)
|
|
12
12
|
.get(`/api/v1/settings/get-setting/${key}`)
|
|
13
|
-
.query(labels ? { labels } : undefined);
|
|
13
|
+
.query(labels ? { labels: labels.map((label: any) => JSON.stringify(label)) } : undefined);
|
|
14
14
|
|
|
15
15
|
return n.reply(response, { value });
|
|
16
16
|
};
|
package/src/index.ts
CHANGED
|
@@ -113,13 +113,13 @@ class SettingsManager {
|
|
|
113
113
|
this.settingsCache.set(cacheKey, waitingToNetwork, this.ttl);
|
|
114
114
|
const networkResponse = await this.network.get(`/api/v1/settings/get-setting/${key}`, {
|
|
115
115
|
params: {
|
|
116
|
-
labels,
|
|
116
|
+
labels: labels.map((label) => JSON.stringify(label)),
|
|
117
117
|
},
|
|
118
118
|
timeout,
|
|
119
119
|
});
|
|
120
120
|
networkValue = networkResponse.data.value;
|
|
121
121
|
} catch (error) {
|
|
122
|
-
logger.error('Cant get setting from network');
|
|
122
|
+
logger.error('Cant get setting from network', { error });
|
|
123
123
|
if (rejectOnFail) {
|
|
124
124
|
throw new CannotGetNetworkValueError();
|
|
125
125
|
}
|