Skip to content

Conversation

@eric-pSAP
Copy link
Contributor

Currently when attempting to UPDATE or PUT into a /content handler, it will overwrite any file existing if the ID is the same as an existing attachment.

@eric-pSAP eric-pSAP marked this pull request as ready for review December 10, 2025 14:01
@eric-pSAP eric-pSAP requested a review from a team as a code owner December 10, 2025 14:01
@eric-pSAP eric-pSAP marked this pull request as draft December 10, 2025 14:35
@eric-pSAP eric-pSAP marked this pull request as ready for review December 11, 2025 12:43
@eric-pSAP eric-pSAP marked this pull request as draft December 11, 2025 15:33
Copy link
Contributor

@hyperspace-insights hyperspace-insights bot left a comment

Choose a reason for hiding this comment

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

The review has been concluded!

PR Bot Information

Version: 1.16.24

  • Event Trigger: issue_comment.created
  • Correlation ID: 0ede29b0-d733-11f0-8487-3deeaddddb31
  • LLM: anthropic--claude-4.5-sonnet

@hyperspace-insights hyperspace-insights bot deleted a comment from KoblerS Dec 12, 2025
eric-pSAP and others added 3 commits December 12, 2025 09:34
Co-authored-by: hyperspace-insights[bot] <209611008+hyperspace-insights[bot]@users.noreply.github.com>
Co-authored-by: hyperspace-insights[bot] <209611008+hyperspace-insights[bot]@users.noreply.github.com>
Co-authored-by: hyperspace-insights[bot] <209611008+hyperspace-insights[bot]@users.noreply.github.com>
eric-pSAP and others added 4 commits December 12, 2025 09:36
Co-authored-by: hyperspace-insights[bot] <209611008+hyperspace-insights[bot]@users.noreply.github.com>
Co-authored-by: hyperspace-insights[bot] <209611008+hyperspace-insights[bot]@users.noreply.github.com>
Co-authored-by: hyperspace-insights[bot] <209611008+hyperspace-insights[bot]@users.noreply.github.com>
@eric-pSAP eric-pSAP marked this pull request as ready for review December 12, 2025 08:37
@eric-pSAP eric-pSAP marked this pull request as draft December 12, 2025 08:48
Removed duplicate call to AttachmentsSrv.put.
@eric-pSAP eric-pSAP marked this pull request as ready for review December 12, 2025 08:49
Removed duplicate comment about blob existence.
Removed redundant check for file existence before throwing an error.
Removed 'attachments' log level from configuration.
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