Merge pull request #22 from Azure-Samples/gk/steps-observaibility

Adds document upload and processing API

Author: thegovind

azure.yaml

@@ -25,6 +25,14 @@ services:
     docker:
       path: ./Dockerfile
       context: .
+      buildArgs:
+        - "VITE_API_BASE_URL=https://caapi-pt7qjgzmrnxgo.politepond-02f6cb2e.eastus2.azurecontainerapps.io/api"
+        - "VITE_AAD_CLIENT_ID=c448ea01-dc6a-4aaa-910d-e6c307db9fc1"
+        - "VITE_AAD_TENANT_ID=56643691-ef04-4208-a672-7ffca73f561c"
+        - "VITE_AAD_USER_FLOW=emailsignflow"
+        - "VITE_AAD_AUTHORITY=https://gkadaptiverag.ciamlogin.com/"
+        - "VITE_AAD_REDIRECT_URI=https://caweb-pt7qjgzmrnxgo.politepond-02f6cb2e.eastus2.azurecontainerapps.io/redirect"
+        - "VITE_API_SCOPE=api://c448ea01-dc6a-4aaa-910d-e6c307db9fc1/access_as_user"
 
 infra:
   provider: bicep
@@ -56,8 +64,4 @@ hooks:
       from app.ingestion.indexer_job import create_search_index
       asyncio.run(create_search_index())
       "
-  prepackage:
-    shell: sh
-    run: |
-      echo "Building frontend..."
-      cd frontend && npm install && npm run build
+

frontend/.dockerignore

@@ -0,0 +1,13 @@
+node_modules
+dist
+package-lock.json
+.env
+.env.local
+.env.development.local
+.env.test.local
+.env.production.local
+npm-debug.log*
+yarn-debug.log*
+yarn-error.log*
+.DS_Store
+*.log 

frontend/Dockerfile

@@ -1,30 +1,55 @@
-FROM node:18-alpine as build
+FROM node:18-alpine AS build
 
 WORKDIR /app
 
 # Copy package files
-COPY package*.json ./
+COPY package.json ./
 
 # Install ALL dependencies (including dev dependencies needed for build)
-RUN npm ci
+RUN npm install
 
 # Copy source code
 COPY . .
 
+# Accept build args for environment variables
+ARG VITE_API_BASE_URL
+ARG VITE_AAD_CLIENT_ID
+ARG VITE_AAD_TENANT_ID
+ARG VITE_AAD_USER_FLOW
+ARG VITE_AAD_AUTHORITY
+ARG VITE_AAD_REDIRECT_URI
+ARG VITE_API_SCOPE
+
+# Set environment variables for the build
+ENV VITE_API_BASE_URL=$VITE_API_BASE_URL
+ENV VITE_AAD_CLIENT_ID=$VITE_AAD_CLIENT_ID
+ENV VITE_AAD_TENANT_ID=$VITE_AAD_TENANT_ID
+ENV VITE_AAD_USER_FLOW=$VITE_AAD_USER_FLOW
+ENV VITE_AAD_AUTHORITY=$VITE_AAD_AUTHORITY
+ENV VITE_AAD_REDIRECT_URI=$VITE_AAD_REDIRECT_URI
+ENV VITE_API_SCOPE=$VITE_API_SCOPE
+
 # Build the application
 RUN npm run build
 
 # Production stage
 FROM nginx:alpine
 
+# Remove the stock config so our template wins
+RUN rm /etc/nginx/conf.d/default.conf
+
 # Copy built assets from build stage
 COPY --from=build /app/dist /usr/share/nginx/html
 
-# Copy nginx configuration template
-COPY nginx.conf.template /etc/nginx/templates/default.conf.template
+# Rename the template so it does not clash with the deleted file
+COPY nginx.conf.template /etc/nginx/templates/api-proxy.conf.template
+
+# Optional banner to show environment variables
+RUN printf '#!/bin/sh\necho "Environment variables:"\nenv | grep -E "(API_|VITE_)"\necho "Starting nginx..."\n' \
+    > /docker-entrypoint.d/00-print-env.sh && chmod +x /docker-entrypoint.d/00-print-env.sh
 
 # Expose port 80
 EXPOSE 80
 
-# Debug environment variables and start nginx
-CMD ["sh", "-c", "echo 'Environment variables:' && env | grep -E '(API_|VITE_)' && echo 'Starting nginx...' && nginx -g 'daemon off;'"]
+# Revert to the normal CMD so the entrypoint renders templates
+CMD ["nginx", "-g", "daemon off;"]

frontend/nginx.conf.template

@@ -1,24 +1,33 @@
 server {
-    listen       80;
-    server_name  localhost;
+    listen 80;
+    server_name localhost;
 
     location / {
-        root   /usr/share/nginx/html;
-        index  index.html index.htm;
+        root /usr/share/nginx/html;
+        index index.html index.htm;
         try_files $uri $uri/ /index.html;
     }
 
-    # API proxy configuration
+    # ---------- proxy every /api/* call to the back‑end Container App ----------
     location /api/ {
-        proxy_pass http://${API_SERVICE_NAME:-api}:8000/;
-        proxy_set_header Host $host;
-        proxy_set_header X-Real-IP $remote_addr;
-        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
+        proxy_pass http://$API_SERVICE_NAME:8000;
+        proxy_set_header Host              $host;
+        proxy_set_header X-Real-IP         $remote_addr;
+        proxy_set_header X-Forwarded-For   $proxy_add_x_forwarded_for;
         proxy_set_header X-Forwarded-Proto $scheme;
+        
+        # SSE-specific configuration
+        proxy_buffering off;
+        proxy_cache off;
+        proxy_set_header Connection '';
+        proxy_http_version 1.1;
+        chunked_transfer_encoding off;
+        
+        # Increase timeout for streaming responses
+        proxy_read_timeout 300s;
+        proxy_send_timeout 300s;
     }
 
-    error_page   500 502 503 504  /50x.html;
-    location = /50x.html {
-        root   /usr/share/nginx/html;
-    }
+    error_page 500 502 503 504 /50x.html;
+    location = /50x.html { root /usr/share/nginx/html; }
 }