From 1fb4b4eb44311bd4375bbeb78a7c252c50e1ed6f Mon Sep 17 00:00:00 2001
From: Camila Belo <camilaibs@gmail.com>
Date: Wed, 4 Aug 2021 12:05:10 +0200
Subject: [PATCH] feat(techdocs-common): lowercase entity triplet path for AWS

Signed-off-by: Camila Belo <camilaibs@gmail.com>
---
 .../src/stages/publish/awsS3.ts               | 20 ++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/packages/techdocs-common/src/stages/publish/awsS3.ts b/packages/techdocs-common/src/stages/publish/awsS3.ts
index 7bdc91ffd4..d08a7d55e7 100644
--- a/packages/techdocs-common/src/stages/publish/awsS3.ts
+++ b/packages/techdocs-common/src/stages/publish/awsS3.ts
@@ -13,7 +13,11 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { Entity, EntityName } from '@backstage/catalog-model';
+import {
+  Entity,
+  EntityName,
+  ENTITY_DEFAULT_NAMESPACE,
+} from '@backstage/catalog-model';
 import { Config } from '@backstage/config';
 import aws, { Credentials } from 'aws-sdk';
 import { ListObjectsV2Output, ManagedUpload } from 'aws-sdk/clients/s3';
@@ -196,8 +200,12 @@ export class AwsS3Publish implements PublisherBase {
           .join(path.posix.sep);
 
         // The / delimiter is intentional since it represents the cloud storage and not the local file system.
-        const entityRootDir = `${entity.metadata.namespace}/${entity.kind}/${entity.metadata.name}`;
-        const destination = `${entityRootDir}/${relativeFilePathPosix}`; // S3 Bucket file relative path
+        const entityRootDir = `${
+          entity.metadata?.namespace ?? ENTITY_DEFAULT_NAMESPACE
+        }/${entity.kind}/${entity.metadata.name}`;
+        const destination = lowerCaseEntityTripletInStoragePath(
+          `${entityRootDir}/${relativeFilePathPosix}`,
+        ); // S3 Bucket file relative path
 
         // Rate limit the concurrent execution of file uploads to batches of 10 (per publish)
         const uploadFile = limiter(() => {
@@ -268,8 +276,10 @@ export class AwsS3Publish implements PublisherBase {
   docsRouter(): express.Handler {
     return async (req, res) => {
       // Decode and trim the leading forward slash
-      // filePath example - /default/Component/documented-component/index.html
-      const filePath = decodeURI(req.path.replace(/^\//, ''));
+      const decodedUri = decodeURI(req.path.replace(/^\//, ''));
+
+      // filePath example - /default/component/documented-component/index.html
+      const filePath = lowerCaseEntityTripletInStoragePath(decodedUri);
 
       // Files with different extensions (CSS, HTML) need to be served with different headers
       const fileExtension = path.extname(filePath);