larryh/corso
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update to attachment.go

Browse Source 
Update for guard checking prior to dereferencing pointer. Failure to do so causes panic on some text cases.
This commit is contained in:
Danny 2023-02-01 23:24:03 -05:00 committed by GitHub
parent 57d37c70ce
commit 0196e3b953
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/internal/connector/exchange/attachment.go
View File

@ -55,8 +55,12 @@ func uploadAttachment(
// item Attachments to be skipped until the completion of Issue #2353 // item Attachments to be skipped until the completion of Issue #2353
if attachmentType == models.ITEM_ATTACHMENTTYPE { if attachmentType == models.ITEM_ATTACHMENTTYPE {
name := ""
if attachment.GetName() != nil{
name := *attachment.GetName()
}
logger.Ctx(ctx).Infow("item attachment uploads are not supported ", logger.Ctx(ctx).Infow("item attachment uploads are not supported ",
"attachment_name", *attachment.GetName(), // TODO: Update to support PII protection "attachment_name", name, // TODO: Update to support PII protection
"attachment_type", attachmentType, "attachment_type", attachmentType,
"internal_item_type", getItemAttachmentItemType(attachment), "internal_item_type", getItemAttachmentItemType(attachment),
"attachment_id", *attachment.GetId(), "attachment_id", *attachment.GetId(),