update about filesWithParentDirs
This commit is contained in:
		
							parent
							
								
									84047d4bcd
								
							
						
					
					
						commit
						d31d2bc74c
					
				|  | @ -87,6 +87,11 @@ The API will iterate over the set of filepaths and for each item | ||||||
|     * check whether the target is whitelisted and if |     * check whether the target is whitelisted and if | ||||||
|       not add the target to the output |       not add the target to the output | ||||||
| 
 | 
 | ||||||
|  | All ancestors of each filepath will also be added to the list, but the previous | ||||||
|  | checks will not be applied to the ancestors. This maintains the current behavior | ||||||
|  | which we believe is needed to maintain correct permissions on the ancestor | ||||||
|  | directories. | ||||||
|  | 
 | ||||||
| ### Open Issues/Questions | ### Open Issues/Questions | ||||||
| 
 | 
 | ||||||
| \<Ignore symlinks targeting whitelisted paths?\> | \<Ignore symlinks targeting whitelisted paths?\> | ||||||
|  | @ -109,7 +114,9 @@ current whitelist logic it is possible for `/some/dir` to be whitelisted but not | ||||||
| ancestors does it make most sense to handle this within the proposed filtering | ancestors does it make most sense to handle this within the proposed filtering | ||||||
| API? | API? | ||||||
| 
 | 
 | ||||||
| Resolution: Not Yet Resolved | Resolution: Resolved | ||||||
|  | 
 | ||||||
|  | Yes, this should be handled in the API | ||||||
| 
 | 
 | ||||||
| \<Should the API handle diff'ing files?\> | \<Should the API handle diff'ing files?\> | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue