fix(server): should use signed url first (#13109)
#### PR Dependency Tree * **PR #13109** 👈 This tree was auto-generated by [Charcoal](https://github.com/danerwilliams/charcoal) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Refactor** * Updated internal handling of comment attachments to improve processing logic. No visible changes to end-user features or workflows. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
@@ -195,7 +195,7 @@ export class WorkspacesController {
|
|||||||
await this.ac.user(user.id).doc(workspaceId, docId).assert('Doc.Read');
|
await this.ac.user(user.id).doc(workspaceId, docId).assert('Doc.Read');
|
||||||
|
|
||||||
const { body, metadata, redirectUrl } =
|
const { body, metadata, redirectUrl } =
|
||||||
await this.commentAttachmentStorage.get(workspaceId, docId, key);
|
await this.commentAttachmentStorage.get(workspaceId, docId, key, true);
|
||||||
|
|
||||||
if (redirectUrl) {
|
if (redirectUrl) {
|
||||||
return res.redirect(redirectUrl);
|
return res.redirect(redirectUrl);
|
||||||
|
|||||||
Reference in New Issue
Block a user