fix(server): return 500 error when user data store is unconfigured

[channi23]

Proposed Changes

Updated the error response in
apps/meteor/server/routes/userDataDownload.ts
from 403 Forbidden to 500 Internal Server Error when the UserDataFiles store is not configured.

Reasoning

A missing UserDataFiles store is a server-side configuration issue, not a permissions problem.
Returning a 500 status with a clear message (User Data Store not configured) more accurately reflects the failure mode and helps administrators diagnose misconfiguration.

Issue(s)

N/A

Steps to Test / Reproduce

1. Run Rocket.Chat.
2. Configure the server such that the UserDataFiles store is not available or unconfigured.
3. Request a user data download.

Before:

• Response status: 403 Forbidden

After:

• Response status: 500 Internal Server Error
• Response body: User Data Store not configured

Additional Notes

This is a small, targeted fix to improve error correctness and debuggability.

Summary by CodeRabbit

• Bug Fixes
  • Improved error handling for user data downloads by providing a clearer error message when the system configuration is incomplete, enhancing troubleshooting experience.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[changeset-bot]

⚠️ No Changeset found

Latest commit: 978cbb6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[coderabbitai]

Walkthrough

The change updates error handling in the user data download route, modifying the HTTP response code from 403 to 500 when the User Data Store is not configured, along with an updated error message.

Changes

Cohort / File(s)	Change Summary
Error Response Code Update apps/meteor/server/routes/userDataDownload.ts	Changed HTTP status from 403 to 500 for missing User Data Store; updated error message from TODO comment to "User Data Store not configured"

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 A hop, a skip, a status code change,

From four-oh-three to five-zero-strange,

When data stores vanish without a trace,

We serve up five-hundred with newfound grace! 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and specifically describes the main change: returning a 500 error instead of 403 when the user data store is unconfigured, which matches the primary modification in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cubic-dev-ai]

No issues found across 1 file

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

apps/meteor/server/routes/userDataDownload.ts (1)

50-51: Consider adding server-side logging for observability.

For better operational visibility, consider logging this configuration error server-side (e.g., using a logger at error level) so administrators can proactively detect and resolve the issue through monitoring rather than waiting for user reports.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between a8a84ab and 978cbb6.

📒 Files selected for processing (1)

• apps/meteor/server/routes/userDataDownload.ts

🧰 Additional context used

📓 Path-based instructions (1)

**/*.{ts,tsx,js}

📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)

**/*.{ts,tsx,js}: Write concise, technical TypeScript/JavaScript with accurate typing in Playwright tests
Avoid code comments in the implementation

Files:

• apps/meteor/server/routes/userDataDownload.ts

🔇 Additional comments (1)

apps/meteor/server/routes/userDataDownload.ts (1)

50-51: Correct status code change and helpful error message.

The change from 403 to 500 is semantically correct—an unconfigured User Data Store is a server configuration issue, not a permissions problem. The descriptive error message aids administrators in diagnosing the issue without exposing sensitive information.