Simplify getting use-ssh-substituter from untrusted users
This commit is contained in:
		
							parent
							
								
									bf4a577a58
								
							
						
					
					
						commit
						42eb4afd7a
					
				
					 2 changed files with 3 additions and 14 deletions
				
			
		|  | @ -111,18 +111,6 @@ void run(Strings args) | ||||||
|     if (args.empty()) |     if (args.empty()) | ||||||
|         throw UsageError("download-via-ssh requires an argument"); |         throw UsageError("download-via-ssh requires an argument"); | ||||||
| 
 | 
 | ||||||
|     Settings::SettingsMap overrides = settings.getOverrides(); |  | ||||||
|     Settings::SettingsMap::iterator use = overrides.find("untrusted-use-ssh-substituter"); |  | ||||||
|     if (use != overrides.end()) { |  | ||||||
|         if (use->second == "true") settings.useSshSubstituter = true; |  | ||||||
|         else if (use->second == "false") settings.useSshSubstituter = false; |  | ||||||
|         else throw Error(format("configuration option `use-ssh-substituter' should be either `true' or `false', not `%1%'") |  | ||||||
|                         % use->second); |  | ||||||
|     } |  | ||||||
| 
 |  | ||||||
|     if (!settings.useSshSubstituter) |  | ||||||
|         return; |  | ||||||
| 
 |  | ||||||
|     if (settings.sshSubstituterHosts.empty()) |     if (settings.sshSubstituterHosts.empty()) | ||||||
|         return; |         return; | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
|  | @ -557,11 +557,12 @@ static void performOp(bool trusted, unsigned int clientVersion, | ||||||
|             for (unsigned int i = 0; i < n; i++) { |             for (unsigned int i = 0; i < n; i++) { | ||||||
|                 string name = readString(from); |                 string name = readString(from); | ||||||
|                 string value = readString(from); |                 string value = readString(from); | ||||||
|                 if (name == "build-timeout") |                 if (name == "build-timeout" || name == "use-ssh-substituter") | ||||||
|                     string2Int(value, settings.buildTimeout); |                     settings.set(name, value); | ||||||
|                 else |                 else | ||||||
|                     settings.set(trusted ? name : "untrusted-" + name, value); |                     settings.set(trusted ? name : "untrusted-" + name, value); | ||||||
|             } |             } | ||||||
|  |             settings.update(); | ||||||
|         } |         } | ||||||
|         startWork(); |         startWork(); | ||||||
|         stopWork(); |         stopWork(); | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue