mirror of
				https://github.com/ansible-collections/community.general.git
				synced 2025-10-24 21:14:00 -07:00 
			
		
		
		
	properly convert inputs to handle bytes/unicode (#53072)
* properly convert inputs to handle bytes/unicode fixes #52186 * Update changelogs/fragments/nmap_bytes_fix.yml Co-Authored-By: bcoca <bcoca@users.noreply.github.com>
This commit is contained in:
		
					parent
					
						
							
								d2bdbadb03
							
						
					
				
			
			
				commit
				
					
						55dc63be3a
					
				
			
		
					 2 changed files with 4 additions and 1 deletions
				
			
		
							
								
								
									
										2
									
								
								changelogs/fragments/nmap_bytes_fix.yml
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										2
									
								
								changelogs/fragments/nmap_bytes_fix.yml
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,2 @@ | |||
| bugfixes: | ||||
|     - convert input into text to ensure valid comparisons in nmap inventory plugin | ||||
|  | @ -57,7 +57,7 @@ from subprocess import Popen, PIPE | |||
| 
 | ||||
| from ansible import constants as C | ||||
| from ansible.errors import AnsibleParserError | ||||
| from ansible.module_utils._text import to_native | ||||
| from ansible.module_utils._text import to_native, to_text | ||||
| from ansible.plugins.inventory import BaseInventoryPlugin, Constructable, Cacheable | ||||
| 
 | ||||
| 
 | ||||
|  | @ -127,6 +127,7 @@ class InventoryModule(BaseInventoryPlugin, Constructable, Cacheable): | |||
|             ip = None | ||||
|             ports = [] | ||||
|             for line in stdout.splitlines(): | ||||
|                 line = to_text(line) | ||||
|                 hits = self.find_host.match(line) | ||||
|                 if hits: | ||||
|                     if host is not None: | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue