fix: Add Accept-Language to API calls when locale is set (#182)

Author: bflorian

lib/smart-app.js

@@ -420,7 +420,8 @@ class SmartApp {
 		const ctx = await this.withContext({
 			installedAppId: auth.installed_app_id,
 			authToken: auth.access_token,
-			refreshToken: auth.refresh_token
+			refreshToken: auth.refresh_token,
+			locale: request.headers ? request.headers['Accept-Language'] : undefined
 		})
 
 		const isa = await ctx.api.installedApps.get(auth.installed_app_id)

lib/util/smart-app-context.js

@@ -115,6 +115,13 @@ module.exports = class SmartAppContext {
 				locationId: this.locationId,
 				installedAppId: this.installedAppId
 			}
+
+			if (this.locale) {
+				config.headers = {
+					'Accept-Language': this.locale
+				}
+			}
+
 			if (app._apiUrl) {
 				config.urlProvider = {
 					baseURL: app._apiUrl,
@@ -158,6 +165,13 @@ module.exports = class SmartAppContext {
 					locationId: this.locationId,
 					installedAppId: this.installedAppId
 				}
+
+				if (this.locale) {
+					config.headers = {
+						'Accept-Language': this.locale
+					}
+				}
+
 				if (app._apiUrl) {
 					config.urlProvider = {
 						baseURL: app._apiUrl,

test/unit/smartapp-context-spec.js

@@ -122,4 +122,18 @@ describe('smartapp-context-spec', () => {
 		assert.equal(params.refreshToken, ctx.refreshToken)
 		assert.equal(params.locale, ctx.event.locale)
 	})
+
+	it('accept-language header in API calls', async () => {
+		const params = {
+			authToken: 'xxx',
+			installedAppId: 'aaa',
+			locationId: 'bbb',
+			locale: 'es',
+			config: {device: 'ccc'}
+		}
+
+		const ctx = await app.withContext(params)
+
+		expect(ctx.api.config.headers['Accept-Language']).toBe('es')
+	})
 })