Skip to content

Conversation

@macabu
Copy link
Contributor

@macabu macabu commented Nov 17, 2025

I noticed this error in the logs:

"error while uploading output to <ts> in slack channel <redacted>: file.upload.v2: filename cannot be empty"

Looking at Slack docs https://docs.slack.dev/messaging/working-with-files/#uploading_files, it requires both file name + file size in bytes as arguments.

@macabu macabu requested review from a team and julienduchesne as code owners November 17, 2025 11:21
@macabu macabu changed the title Slack: Fix uploading test results to message fix(slack): Upload failed test results to message Nov 17, 2025
@macabu macabu changed the title fix(slack): Upload failed test results to message fix(slack): Upload test results to message Nov 17, 2025
@macabu macabu changed the title fix(slack): Upload test results to message fix(slack): upload test results to message Nov 17, 2025
Copy link
Contributor

@zerok zerok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 😄

@zerok zerok merged commit 4c473e6 into main Nov 17, 2025
12 of 15 checks passed
@zerok zerok deleted the macabu/add-slack-filename-to-uploads branch November 17, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants