@atlaskit/media-client 28.0.6 → 28.0.8

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @atlaskit/media-client
 
+## 28.0.8
+
+### Patch Changes
+
+- [#158161](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/158161)
+  [`f092b9d2f949d`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/f092b9d2f949d) -
+  clean up LD Media CDN FFs
+
+## 28.0.7
+
+### Patch Changes
+
+- [#158481](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/158481)
+  [`a6efbf9ceb2c9`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/a6efbf9ceb2c9) -
+  Removed nonMediaErrors due to incorrectly logging serverUnauthorised
+
 ## 28.0.6
 
 ### Patch Changes

package/dist/cjs/client/media-store/MediaStore.js

@@ -42,7 +42,7 @@ var jsonHeaders = {
 };
 var cdnFeatureFlag = function cdnFeatureFlag(endpoint) {
   var result = endpoint;
-  if ((0, _platformFeatureFlags.fg)('platform.media-cdn-delivery') || (0, _platformFeatureFlags.fg)('platform_media_cdn_delivery')) {
+  if ((0, _platformFeatureFlags.fg)('platform_media_cdn_delivery')) {
     if ((0, _isCommercial.isCommercial)()) {
       result += '/cdn';
     }

package/dist/cjs/models/artifacts.js

@@ -9,7 +9,7 @@ var _isCommercial = require("../utils/isCommercial");
 var getArtifactUrl = exports.getArtifactUrl = function getArtifactUrl(artifacts, prop) {
   var _artifacts$prop, _artifacts$prop3;
   // eslint-disable-next-line @atlaskit/platform/no-preconditioning
-  if ((0, _isCommercial.isCommercial)() && ((0, _platformFeatureFlags.fg)('platform.media-cdn-delivery') || (0, _platformFeatureFlags.fg)('platform_media_cdn_delivery')) && (_artifacts$prop = artifacts[prop]) !== null && _artifacts$prop !== void 0 && _artifacts$prop.url) {
+  if ((0, _isCommercial.isCommercial)() && (0, _platformFeatureFlags.fg)('platform_media_cdn_delivery') && (_artifacts$prop = artifacts[prop]) !== null && _artifacts$prop !== void 0 && _artifacts$prop.url) {
     var _artifacts$prop2;
     return "".concat((_artifacts$prop2 = artifacts[prop]) === null || _artifacts$prop2 === void 0 ? void 0 : _artifacts$prop2.url, "/cdn");
   }

package/dist/cjs/utils/mediaCdn.js

@@ -23,7 +23,7 @@ function mapToMediaCdnUrl(url, token) {
     return url;
   }
   // eslint-disable-next-line @atlaskit/platform/no-preconditioning
-  if (((0, _platformFeatureFlags.fg)('platform.media-cdn-delivery') || (0, _platformFeatureFlags.fg)('platform_media_cdn_delivery')) && ((0, _platformFeatureFlags.fg)('platform.media-cdn-single-host') || (0, _platformFeatureFlags.fg)('platform_media_cdn_single_host'))) {
+  if ((0, _platformFeatureFlags.fg)('platform_media_cdn_delivery') && (0, _platformFeatureFlags.fg)('platform_media_cdn_single_host')) {
     try {
       var parsedUrl = new URL(url);
       var cdnHost = MEDIA_CDN_MAP[parsedUrl.host];

package/dist/cjs/utils/request/helpers.js

@@ -285,15 +285,8 @@ function createRequestErrorReason(statusCode) {
 }
 function createRequestErrorFromResponse(metadata, response) {
   var statusCode = response.status;
-
-  // with media-cdn single host feature on ('platform.media-cdn-single-host'), media will make a call to Cloudfront before dt-api-filestore, where 'x-media-region' and 'x-media-env' are set. The absence of these headers indicates a Cloudfront error.
-  var _extractMediaHeaders = extractMediaHeaders(response),
-    mediaRegion = _extractMediaHeaders.mediaRegion,
-    mediaEnv = _extractMediaHeaders.mediaEnv;
-  var reason = mediaRegion === 'unknown' || mediaEnv === 'unknown' ? 'nonMediaError' : createRequestErrorReason(statusCode);
-  return new _errors.RequestError(reason, _objectSpread(_objectSpread({}, metadata), {}, {
-    mediaRegion: mediaRegion,
-    mediaEnv: mediaEnv,
+  var reason = createRequestErrorReason(statusCode);
+  return new _errors.RequestError(reason, _objectSpread(_objectSpread(_objectSpread({}, metadata), extractMediaHeaders(response)), {}, {
     statusCode: statusCode
   }));
 }

package/dist/es2019/client/media-store/MediaStore.js

@@ -31,7 +31,7 @@ const jsonHeaders = {
 };
 const cdnFeatureFlag = endpoint => {
   let result = endpoint;
-  if (fg('platform.media-cdn-delivery') || fg('platform_media_cdn_delivery')) {
+  if (fg('platform_media_cdn_delivery')) {
     if (isCommercial()) {
       result += '/cdn';
     }

package/dist/es2019/models/artifacts.js

@@ -3,7 +3,7 @@ import { isCommercial } from '../utils/isCommercial';
 export const getArtifactUrl = (artifacts, prop) => {
   var _artifacts$prop, _artifacts$prop3;
   // eslint-disable-next-line @atlaskit/platform/no-preconditioning
-  if (isCommercial() && (fg('platform.media-cdn-delivery') || fg('platform_media_cdn_delivery')) && (_artifacts$prop = artifacts[prop]) !== null && _artifacts$prop !== void 0 && _artifacts$prop.url) {
+  if (isCommercial() && fg('platform_media_cdn_delivery') && (_artifacts$prop = artifacts[prop]) !== null && _artifacts$prop !== void 0 && _artifacts$prop.url) {
     var _artifacts$prop2;
     return `${(_artifacts$prop2 = artifacts[prop]) === null || _artifacts$prop2 === void 0 ? void 0 : _artifacts$prop2.url}/cdn`;
   }

package/dist/es2019/utils/mediaCdn.js

@@ -16,7 +16,7 @@ export function mapToMediaCdnUrl(url, token) {
     return url;
   }
   // eslint-disable-next-line @atlaskit/platform/no-preconditioning
-  if ((fg('platform.media-cdn-delivery') || fg('platform_media_cdn_delivery')) && (fg('platform.media-cdn-single-host') || fg('platform_media_cdn_single_host'))) {
+  if (fg('platform_media_cdn_delivery') && fg('platform_media_cdn_single_host')) {
     try {
       const parsedUrl = new URL(url);
       const cdnHost = MEDIA_CDN_MAP[parsedUrl.host];

package/dist/es2019/utils/request/helpers.js

@@ -183,17 +183,10 @@ export function createRequestErrorFromResponse(metadata, response) {
   const {
     status: statusCode
   } = response;
-
-  // with media-cdn single host feature on ('platform.media-cdn-single-host'), media will make a call to Cloudfront before dt-api-filestore, where 'x-media-region' and 'x-media-env' are set. The absence of these headers indicates a Cloudfront error.
-  const {
-    mediaRegion,
-    mediaEnv
-  } = extractMediaHeaders(response);
-  const reason = mediaRegion === 'unknown' || mediaEnv === 'unknown' ? 'nonMediaError' : createRequestErrorReason(statusCode);
+  const reason = createRequestErrorReason(statusCode);
   return new RequestError(reason, {
     ...metadata,
-    mediaRegion,
-    mediaEnv,
+    ...extractMediaHeaders(response),
     statusCode
   });
 }

package/dist/esm/client/media-store/MediaStore.js

@@ -36,7 +36,7 @@ var jsonHeaders = {
 };
 var cdnFeatureFlag = function cdnFeatureFlag(endpoint) {
   var result = endpoint;
-  if (fg('platform.media-cdn-delivery') || fg('platform_media_cdn_delivery')) {
+  if (fg('platform_media_cdn_delivery')) {
     if (isCommercial()) {
       result += '/cdn';
     }

package/dist/esm/models/artifacts.js

@@ -3,7 +3,7 @@ import { isCommercial } from '../utils/isCommercial';
 export var getArtifactUrl = function getArtifactUrl(artifacts, prop) {
   var _artifacts$prop, _artifacts$prop3;
   // eslint-disable-next-line @atlaskit/platform/no-preconditioning
-  if (isCommercial() && (fg('platform.media-cdn-delivery') || fg('platform_media_cdn_delivery')) && (_artifacts$prop = artifacts[prop]) !== null && _artifacts$prop !== void 0 && _artifacts$prop.url) {
+  if (isCommercial() && fg('platform_media_cdn_delivery') && (_artifacts$prop = artifacts[prop]) !== null && _artifacts$prop !== void 0 && _artifacts$prop.url) {
     var _artifacts$prop2;
     return "".concat((_artifacts$prop2 = artifacts[prop]) === null || _artifacts$prop2 === void 0 ? void 0 : _artifacts$prop2.url, "/cdn");
   }

package/dist/esm/utils/mediaCdn.js

@@ -16,7 +16,7 @@ export function mapToMediaCdnUrl(url, token) {
     return url;
   }
   // eslint-disable-next-line @atlaskit/platform/no-preconditioning
-  if ((fg('platform.media-cdn-delivery') || fg('platform_media_cdn_delivery')) && (fg('platform.media-cdn-single-host') || fg('platform_media_cdn_single_host'))) {
+  if (fg('platform_media_cdn_delivery') && fg('platform_media_cdn_single_host')) {
     try {
       var parsedUrl = new URL(url);
       var cdnHost = MEDIA_CDN_MAP[parsedUrl.host];

package/dist/esm/utils/request/helpers.js

@@ -263,15 +263,8 @@ export function createRequestErrorReason(statusCode) {
 }
 export function createRequestErrorFromResponse(metadata, response) {
   var statusCode = response.status;
-
-  // with media-cdn single host feature on ('platform.media-cdn-single-host'), media will make a call to Cloudfront before dt-api-filestore, where 'x-media-region' and 'x-media-env' are set. The absence of these headers indicates a Cloudfront error.
-  var _extractMediaHeaders = extractMediaHeaders(response),
-    mediaRegion = _extractMediaHeaders.mediaRegion,
-    mediaEnv = _extractMediaHeaders.mediaEnv;
-  var reason = mediaRegion === 'unknown' || mediaEnv === 'unknown' ? 'nonMediaError' : createRequestErrorReason(statusCode);
-  return new RequestError(reason, _objectSpread(_objectSpread({}, metadata), {}, {
-    mediaRegion: mediaRegion,
-    mediaEnv: mediaEnv,
+  var reason = createRequestErrorReason(statusCode);
+  return new RequestError(reason, _objectSpread(_objectSpread(_objectSpread({}, metadata), extractMediaHeaders(response)), {}, {
     statusCode: statusCode
   }));
 }

package/dist/types/models/errors/types.d.ts

@@ -1,4 +1,4 @@
-export type MediaClientErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError' | 'nonMediaError' | 'failedAuthProvider' | 'tokenExpired' | 'missingInitialAuth' | 'emptyAuth' | 'authProviderTimedOut' | 'invalidFileId' | 'emptyItems' | 'zeroVersionFile' | 'emptyFileName' | 'pollingMaxAttemptsExceeded' | 'fileSizeExceedsLimit' | 'deprecatedEndpoint';
+export type MediaClientErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError' | 'failedAuthProvider' | 'tokenExpired' | 'missingInitialAuth' | 'emptyAuth' | 'authProviderTimedOut' | 'invalidFileId' | 'emptyItems' | 'zeroVersionFile' | 'emptyFileName' | 'pollingMaxAttemptsExceeded' | 'fileSizeExceedsLimit' | 'deprecatedEndpoint';
 export interface MediaClientErrorAttributes {
     reason: MediaClientErrorReason;
 }

package/dist/types/utils/request/types.d.ts

@@ -35,7 +35,7 @@ export type CreateUrlOptions = {
     readonly params?: RequestParams;
     readonly auth?: Auth;
 };
-export type RequestErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'nonMediaError' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError';
+export type RequestErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError';
 export type RequestErrorMetadata = RequestMetadata & {
     readonly attempts?: number;
     readonly clientExhaustedRetries?: boolean;

package/dist/types-ts4.5/models/errors/types.d.ts

@@ -1,4 +1,4 @@
-export type MediaClientErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError' | 'nonMediaError' | 'failedAuthProvider' | 'tokenExpired' | 'missingInitialAuth' | 'emptyAuth' | 'authProviderTimedOut' | 'invalidFileId' | 'emptyItems' | 'zeroVersionFile' | 'emptyFileName' | 'pollingMaxAttemptsExceeded' | 'fileSizeExceedsLimit' | 'deprecatedEndpoint';
+export type MediaClientErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError' | 'failedAuthProvider' | 'tokenExpired' | 'missingInitialAuth' | 'emptyAuth' | 'authProviderTimedOut' | 'invalidFileId' | 'emptyItems' | 'zeroVersionFile' | 'emptyFileName' | 'pollingMaxAttemptsExceeded' | 'fileSizeExceedsLimit' | 'deprecatedEndpoint';
 export interface MediaClientErrorAttributes {
     reason: MediaClientErrorReason;
 }

package/dist/types-ts4.5/utils/request/types.d.ts

@@ -35,7 +35,7 @@ export type CreateUrlOptions = {
     readonly params?: RequestParams;
     readonly auth?: Auth;
 };
-export type RequestErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'nonMediaError' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError';
+export type RequestErrorReason = 'clientOffline' | 'clientAbortedRequest' | 'clientTimeoutRequest' | 'serverInvalidBody' | 'serverBadRequest' | 'serverUnauthorized' | 'serverForbidden' | 'serverNotFound' | 'serverRateLimited' | 'serverInternalError' | 'serverBadGateway' | 'serverUnexpectedError';
 export type RequestErrorMetadata = RequestMetadata & {
     readonly attempts?: number;
     readonly clientExhaustedRetries?: boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/media-client",
-	"version": "28.0.6",
+	"version": "28.0.8",
 	"description": "Media API Web Client Library",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -81,15 +81,9 @@
 		}
 	},
 	"platform-feature-flags": {
-		"platform.media-cdn-delivery": {
-			"type": "boolean"
-		},
 		"platform.media-card-performance-observer_a803k": {
 			"type": "boolean"
 		},
-		"platform.media-cdn-single-host": {
-			"type": "boolean"
-		},
 		"platform_media_cdn_single_host": {
 			"type": "boolean"
 		},