fix(server): remove crippled isLocaleAcceptable functionality (#1793)…

…, r=@vbudhram, @rfk
mozilla · Apr 5, 2017 · 748fcee · 748fcee
1 parent d67d395
commit 748fcee
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 20 deletions.
diff --git a/lib/metrics/events.js b/lib/metrics/events.js
@@ -152,7 +152,7 @@ module.exports = log => {
 
     return request.gatherMetricsContext({
       event: event,
-      locale: marshallLocale(request),
+      locale: request.app && request.app.locale,
       uid: coalesceUid(optionalData, request),
       userAgent: request.headers['user-agent']
     }).then(data => {
@@ -183,12 +183,6 @@ function optionallySetService (data, request) {
     (request.query && request.query.service)
 }
 
-function marshallLocale (request) {
-  if (request.app && request.app.locale) {
-    return `${request.app.locale}${request.app.isLocaleAcceptable ? '' : '.default'}`
-  }
-}
-
 function coalesceUid (data, request) {
   if (data && data.uid) {
     return data.uid

diff --git a/lib/server.js b/lib/server.js
@@ -286,9 +286,7 @@ function create(log, error, config, routes, db, translator) {
 
       const acceptLanguage = trimLocale(request.headers['accept-language'])
       request.app.acceptLanguage = acceptLanguage
-      const locale = translator.getLocale(acceptLanguage)
-      request.app.locale = locale
-      request.app.isLocaleAcceptable = isLocaleAcceptable(locale, acceptLanguage)
+      request.app.locale = translator.getLocale(acceptLanguage)
 
       if (request.headers.authorization) {
         // Log some helpful details for debugging authentication problems.
@@ -345,10 +343,6 @@ function create(log, error, config, routes, db, translator) {
   return server
 }
 
-function isLocaleAcceptable (locale, acceptLanguage) {
-  return RegExp(`^(?:.+, *)*${locale}(?:[,-].+)*$`, 'i').test(acceptLanguage)
-}
-
 module.exports = {
   create: create,
   // Functions below exported for testing

diff --git a/test/local/lib/server.js b/test/local/lib/server.js
@@ -112,7 +112,6 @@ describe('lib/server', () => {
           assert.ok(args[1])
           assert.equal(args[1].path, '/account/create')
           assert.equal(args[1].app.locale, 'en')
-          assert.equal(args[1].app.isLocaleAcceptable, true)
         })
 
         it('called log.summary correctly', () => {
@@ -150,7 +149,6 @@ describe('lib/server', () => {
           assert.equal(log.begin.callCount, 1)
           const args = log.begin.args[0]
           assert.equal(args[1].app.locale, 'en')
-          assert.equal(args[1].app.isLocaleAcceptable, false)
         })
 
         it('called log.summary once', () => {

diff --git a/test/local/metrics/events.js b/test/local/metrics/events.js
@@ -170,7 +170,6 @@ describe('metrics/events', () => {
     const metricsContext = mocks.mockMetricsContext()
     const request = {
       app: {
-        isLocaleAcceptable: false,
         locale: 'en'
       },
       auth: {
@@ -204,7 +203,7 @@ describe('metrics/events', () => {
           flow_id: 'bar',
           flow_time: 1000,
           flowCompleteSignal: 'account.signed',
-          locale: 'en.default',
+          locale: 'en',
           time,
           uid: 'deadbeef',
           userAgent: 'foo'
@@ -226,7 +225,6 @@ describe('metrics/events', () => {
     const metricsContext = mocks.mockMetricsContext()
     const request = {
       app: {
-        isLocaleAcceptable: true,
         locale: 'fr'
       },
       clearMetricsContext: metricsContext.clear,

diff --git a/test/mocks.js b/test/mocks.js
@@ -355,7 +355,6 @@ function mockRequest (data) {
     app: {
       acceptLanguage: 'en-US',
       clientAddress: data.clientAddress || '63.245.221.32',
-      isLocaleAcceptable: true,
       locale: data.locale || 'en-US'
     },
     auth: {