mirror of
				https://github.com/optim-enterprises-bv/vault.git
				synced 2025-10-31 02:28:09 +00:00 
			
		
		
		
	MongoDB - Fix write_concern param (#18546)
* fix writeconcern defaulting to majority * add changelog * restart CI tests * fix tests * add package
This commit is contained in:
		 Milena Zlaticanin
					Milena Zlaticanin
				
			
				
					committed by
					
						 GitHub
						GitHub
					
				
			
			
				
	
			
			
			 GitHub
						GitHub
					
				
			
						parent
						
							4a6a41e591
						
					
				
				
					commit
					159b60a181
				
			
							
								
								
									
										3
									
								
								changelog/18546.txt
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										3
									
								
								changelog/18546.txt
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,3 @@ | |||||||
|  | ```release-note:bug | ||||||
|  | database/mongodb: Fix writeConcern set to be applied to any query made on the database | ||||||
|  | ``` | ||||||
| @@ -210,9 +210,19 @@ func (m *MongoDB) DeleteUser(ctx context.Context, req dbplugin.DeleteUserRequest | |||||||
| 		db = "admin" | 		db = "admin" | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
|  | 	// Set the write concern. The default is majority. | ||||||
|  | 	writeConcern := writeconcern.New(writeconcern.WMajority()) | ||||||
|  | 	opts, err := m.getWriteConcern() | ||||||
|  | 	if err != nil { | ||||||
|  | 		return dbplugin.DeleteUserResponse{}, err | ||||||
|  | 	} | ||||||
|  | 	if opts != nil { | ||||||
|  | 		writeConcern = opts.WriteConcern | ||||||
|  | 	} | ||||||
|  |  | ||||||
| 	dropUserCmd := &dropUserCommand{ | 	dropUserCmd := &dropUserCommand{ | ||||||
| 		Username:     req.Username, | 		Username:     req.Username, | ||||||
| 		WriteConcern: writeconcern.New(writeconcern.WMajority()), | 		WriteConcern: writeConcern, | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	err = m.runCommandWithRetry(ctx, db, dropUserCmd) | 	err = m.runCommandWithRetry(ctx, db, dropUserCmd) | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user