Merge pull request #321 from microsoftgraph/andrueastman/fixRepeatedS…

…lices

fix: repeated slice uploading on large file upload task

authentication/azure_identity_access_token_provider.go

@@ -31,7 +31,7 @@ func NewAzureIdentityAccessTokenProviderWithScopesAndValidHostsAndObservabilityO
 	if err != nil {
 		return nil, err
 	}
-	if len(scopes) == 0 {
+	if len(validHosts) == 0 {
 		base.GetAllowedHostsValidator().SetAllowedHosts([]string{"graph.microsoft.com", "graph.microsoft.us", "dod-graph.microsoft.us", "graph.microsoft.de", "microsoftgraph.chinacloudapi.cn", "canary.graph.microsoft.com"})
 	}
 	result := &AzureIdentityAccessTokenProvider{

fileuploader/large_file_upload_task.go

@@ -3,13 +3,14 @@ package fileuploader
 import (
 	"context"
 	"errors"
-	abstractions "github.com/microsoft/kiota-abstractions-go"
-	"github.com/microsoft/kiota-abstractions-go/serialization"
 	"io"
 	"os"
 	"strconv"
 	"strings"
 	"time"
+
+	abstractions "github.com/microsoft/kiota-abstractions-go"
+	"github.com/microsoft/kiota-abstractions-go/serialization"
 )
 
 type LargeFileUploadTask[T serialization.Parsable] interface {
@@ -140,6 +141,8 @@ func (l *largeFileUploadTask[T]) uploadWithRetry(slice uploadSlice[T], maxRetry
 			}
 			// backoff before retrying
 			time.Sleep(time.Duration(retry) * time.Second)
+		} else {
+			return parseable, location, err // return the result as the upload was successful
 		}
 		retry++
 	}