GODRIER-2696 Use default revision for skip num

prestonvasquez · May 10, 2024 · 49e44f8 · 49e44f8
1 parent c94c4e6
commit 49e44f8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/mongo/gridfs_bucket.go b/mongo/gridfs_bucket.go
@@ -197,11 +197,8 @@ func (b *GridFSBucket) OpenDownloadStreamByName(
 		return nil, fmt.Errorf("failed to construct arguments from options: %w", err)
 	}
 
-	var numSkip int32 = -1
-
-	if args.Revision == nil {
-		args.Revision = &options.DefaultRevision
-	} else {
+	numSkip := options.DefaultRevision
+	if args.Revision != nil {
 		numSkip = *args.Revision
 	}
 

diff --git a/mongo/options/clientoptions.go b/mongo/options/clientoptions.go
@@ -517,8 +517,6 @@ func (c *ClientOptions) GetURI() string {
 // error found.
 func (c *ClientOptions) Validate() error {
 	args, err := getArgs[ClientArgs](c)
-	//return err
-
 	if err != nil {
 		return err
 	}