Revert "fix!: error codes (#224)" (#243)

This reverts commit d271e3c.

Author: shazron

lib/AdobeState.js

@@ -112,25 +112,20 @@ async function handleResponse (response, params) {
   const copyParams = cloneDeep(params)
   copyParams.requestId = response.headers.get(REQUEST_ID_HEADER)
 
-  const body = await response.text()
-  if (response.status < 500) {
-    switch (response.status) {
-      case 404:
-        return response // 404s are not an error
-      case 401:
-        return logAndThrow(new codes.ERROR_UNAUTHORIZED({ messageValues: [body], sdkDetails: copyParams }))
-      case 403:
-        return logAndThrow(new codes.ERROR_FORBIDDEN({ messageValues: [body], sdkDetails: copyParams }))
-      case 413:
-        return logAndThrow(new codes.ERROR_PAYLOAD_TOO_LARGE({ messageValues: [body], sdkDetails: copyParams }))
-      case 429:
-        return logAndThrow(new codes.ERROR_REQUEST_RATE_TOO_HIGH({ messageValues: [body], sdkDetails: copyParams }))
-      default: // other 4xx errors
-        return logAndThrow(new codes.ERROR_BAD_REQUEST({ messageValues: [body], sdkDetails: copyParams }))
-    }
+  switch (response.status) {
+    case 404:
+      return response
+    case 401:
+      return logAndThrow(new codes.ERROR_UNAUTHORIZED({ messageValues: ['State service'], sdkDetails: copyParams }))
+    case 403:
+      return logAndThrow(new codes.ERROR_BAD_CREDENTIALS({ messageValues: ['State service'], sdkDetails: copyParams }))
+    case 413:
+      return logAndThrow(new codes.ERROR_PAYLOAD_TOO_LARGE({ messageValues: ['State service'], sdkDetails: copyParams }))
+    case 429:
+      return logAndThrow(new codes.ERROR_REQUEST_RATE_TOO_HIGH({ sdkDetails: copyParams }))
+    default: // 500 errors
+      return logAndThrow(new codes.ERROR_INTERNAL({ messageValues: [`unexpected response from State service with status: ${response.status} body: ${await response.text()}`], sdkDetails: copyParams }))
   }
-  // 500 errors
-  return logAndThrow(new codes.ERROR_INTERNAL({ messageValues: [`unexpected response from State service with body: ${body}`], sdkDetails: copyParams }))
 }
 
 /** @private */

lib/StateError.js

@@ -56,10 +56,11 @@ E('ERROR_INTERNAL', '%s')
 E('ERROR_BAD_REQUEST', '%s')
 E('ERROR_BAD_ARGUMENT', '%s')
 E('ERROR_UNEXPECTED_NOT_FOUND', '%s')
-E('ERROR_UNAUTHORIZED', '%s')
-E('ERROR_FORBIDDEN', '%s')
-E('ERROR_PAYLOAD_TOO_LARGE', '%s')
-E('ERROR_REQUEST_RATE_TOO_HIGH', '%s')
+E('ERROR_UNKNOWN_PROVIDER', '%s')
+E('ERROR_UNAUTHORIZED', 'you are not authorized to access %s')
+E('ERROR_BAD_CREDENTIALS', 'cannot access %s, make sure your credentials are valid')
+E('ERROR_PAYLOAD_TOO_LARGE', 'key, value or request payload is too large')
+E('ERROR_REQUEST_RATE_TOO_HIGH', 'Request rate too high. Please retry after sometime.')
 
 // eslint-disable-next-line jsdoc/require-jsdoc
 function logAndThrow (e) {

test/AdobeState.test.js

@@ -271,69 +271,55 @@ describe('put', () => {
     const key = 'some-key'
     const value = 'some-value'
 
-    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(401, 'myerror'))
+    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(401))
     await expect(store.put(key, value)).rejects.toThrow(
       expect.objectContaining({
         sdkDetails: expect.objectContaining({
           requestId: 'fake-req-id'
         }),
-        message: '[AdobeStateLib:ERROR_UNAUTHORIZED] myerror'
+        message: '[AdobeStateLib:ERROR_UNAUTHORIZED] you are not authorized to access State service'
       }))
   })
 
   test('coverage: 403 error', async () => {
     const key = 'some-key'
     const value = 'some-value'
 
-    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(403, 'myerror'))
+    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(403))
     await expect(store.put(key, value)).rejects.toThrow(
       expect.objectContaining({
         sdkDetails: expect.objectContaining({
           requestId: 'fake-req-id'
         }),
-        message: '[AdobeStateLib:ERROR_FORBIDDEN] myerror'
+        message: '[AdobeStateLib:ERROR_BAD_CREDENTIALS] cannot access State service, make sure your credentials are valid'
       }))
   })
 
   test('coverage: 413 error', async () => {
     const key = 'some-key'
     const value = 'some-value'
 
-    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(413, 'myerror'))
+    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(413))
     await expect(store.put(key, value)).rejects.toThrow(
       expect.objectContaining({
         sdkDetails: expect.objectContaining({
           requestId: 'fake-req-id'
         }),
-        message: '[AdobeStateLib:ERROR_PAYLOAD_TOO_LARGE] myerror'
+        message: '[AdobeStateLib:ERROR_PAYLOAD_TOO_LARGE] key, value or request payload is too large State service'
       }))
   })
 
   test('coverage: 429 error', async () => {
     const key = 'some-key'
     const value = 'some-value'
 
-    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(429, 'myerror'))
+    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(429))
     await expect(store.put(key, value)).rejects.toThrow(
       expect.objectContaining({
         sdkDetails: expect.objectContaining({
           requestId: 'fake-req-id'
         }),
-        message: '[AdobeStateLib:ERROR_REQUEST_RATE_TOO_HIGH] myerror'
-      }))
-  })
-  test('coverage: unknown user error', async () => {
-    const key = 'some-key'
-    const value = 'some-value'
-    const responseBody = 'error: this is the response body'
-
-    mockExponentialBackoff.mockResolvedValue(wrapInFetchError(400, responseBody))
-    await expect(store.put(key, value)).rejects.toThrow(
-      expect.objectContaining({
-        sdkDetails: expect.objectContaining({
-          requestId: 'fake-req-id'
-        }),
-        message: `[AdobeStateLib:ERROR_BAD_REQUEST] ${responseBody}`
+        message: '[AdobeStateLib:ERROR_REQUEST_RATE_TOO_HIGH] Request rate too high. Please retry after sometime.'
       }))
   })
 
@@ -348,7 +334,7 @@ describe('put', () => {
         sdkDetails: expect.objectContaining({
           requestId: 'fake-req-id'
         }),
-        message: `[AdobeStateLib:ERROR_INTERNAL] unexpected response from State service with body: ${responseBody}`
+        message: `[AdobeStateLib:ERROR_INTERNAL] unexpected response from State service with status: 500 body: ${responseBody}`
       }))
   })