Skip to content

chore(release): staging to production - 2026.05.20 - #1080

Merged
ct3685 merged 1 commit into
productionfrom
staging
May 20, 2026
Merged

chore(release): staging to production - 2026.05.20#1080
ct3685 merged 1 commit into
productionfrom
staging

Conversation

@github-actions

Copy link
Copy Markdown

🚀 Release: Staging to Production

Release Date: 2026-05-20

Changes in this release


This PR is automatically created/updated when commits are pushed to staging.
Merging this PR will trigger the release workflow to create a new GitHub release.

…eys absent (#1079)

## Problem

`csvParserService.createCsvParseRun` was building an inline `S3Client`
with:

```ts
credentials: {
    accessKeyId: process.env.S3_STORAGE_ACCESS_KEY_ID ?? '',
    secretAccessKey: process.env.S3_STORAGE_SECRET_ACCESS_KEY ?? ''
}
```

Passing an empty string is **not** the same as omitting `credentials`.
The AWS SDK v3 sees a credential object with an empty AKID and
immediately throws:

> The authorization header is malformed; a non-empty Access Key (AKID)
must be provided in the credential.

This breaks any deployment that relies on IAM instance roles, ECS task
roles, or Copilot-injected credentials rather than explicit env vars.

## Fix

Follow the same pattern used by every other S3 client in the repo
(`chatflow-storage`, `video-generator`, `dalle-image`,
`dalle-image-upload`, logger):

- Only attach `credentials` when **both** env vars are non-empty
- Omit them otherwise → AWS SDK falls through to its default credential
chain (IAM roles, ECS/Copilot task roles, `~/.aws/credentials`, etc.)
- Also added `S3_ENDPOINT_URL` / `forcePathStyle` support for
consistency with the rest of the codebase

## Files changed

- `packages/server/src/services/csv-parser/index.ts`

## Test plan
- [ ] Deploy to an environment using IAM/Copilot credentials (no
explicit `S3_STORAGE_ACCESS_KEY_ID`) — CSV upload should succeed
- [ ] Deploy with explicit `S3_STORAGE_ACCESS_KEY_ID` /
`S3_STORAGE_SECRET_ACCESS_KEY` — still works as before
@vercel

vercel Bot commented May 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
answerai-docs Building Building Preview May 20, 2026 4:07pm
the-answerai Building Building Preview May 20, 2026 4:07pm

Request Review

@ct3685
ct3685 merged commit 46eab93 into production May 20, 2026
7 of 8 checks passed
@maxtechera
maxtechera temporarily deployed to staging - aai-unified2-flowise-moonstruck May 20, 2026 16:07 — with Render Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants