Skip to content

feat(playstation): Upload non-prosperodmp attachments#5673

Open
elramen wants to merge 3 commits intomasterfrom
playstation
Open

feat(playstation): Upload non-prosperodmp attachments#5673
elramen wants to merge 3 commits intomasterfrom
playstation

Conversation

@elramen
Copy link
Member

@elramen elramen commented Feb 27, 2026

Stream non-prosperodmp attachments to objectstore instead of adding them to the envelope. By not reading attachments into memory, they are not limited in size to max_attachment_size, allowing customers to send large attachments such as memory dumps to the playstation endpoint.

Stream non-prosperodmp attachments to objectstore instead of adding them to the envelope. By not reading attachments into memory, they are not limited to max_attachment_size, allowing customers to send large attachments such as memory dumps to the playstation endpoint.
@elramen elramen requested a review from a team as a code owner February 27, 2026 17:44
Comment on lines +254 to +258
if let Ok(location) = location.and_then(|l| l.into_header_value()) {
item.set_payload(
ContentType::AttachmentRef,
location.as_bytes().to_owned(),
);
Copy link
Member Author

Choose a reason for hiding this comment

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

@jjbayer Should we emit an outcome here if the upload fails? Or is that already taken care of elsewhere?

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

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.

1 participant