fix(log-repo): use the original form of the repo url to remove the need to mask credentials (#2459)
fixes #2449
This commit is contained in:
		
							parent
							
								
									17d60d310e
								
							
						
					
					
						commit
						58a226f29c
					
				
							
								
								
									
										3
									
								
								index.js
									
									
									
									
									
								
							
							
						
						
									
										3
									
								
								index.js
									
									
									
									
									
								
							| @ -75,7 +75,7 @@ async function run(context, plugins) { | |||||||
|   } |   } | ||||||
| 
 | 
 | ||||||
|   logger[options.dryRun ? 'warn' : 'success']( |   logger[options.dryRun ? 'warn' : 'success']( | ||||||
|     `Run automated release from branch ${ciBranch} on repository ${options.repositoryUrl}${ |     `Run automated release from branch ${ciBranch} on repository ${options.originalRepositoryURL}${ | ||||||
|       options.dryRun ? ' in dry-run mode' : '' |       options.dryRun ? ' in dry-run mode' : '' | ||||||
|     }` |     }` | ||||||
|   ); |   ); | ||||||
| @ -263,6 +263,7 @@ module.exports = async (cliOptions = {}, {cwd = process.cwd(), env = process.env | |||||||
|   context.logger.log(`Running ${pkg.name} version ${pkg.version}`); |   context.logger.log(`Running ${pkg.name} version ${pkg.version}`); | ||||||
|   try { |   try { | ||||||
|     const {plugins, options} = await getConfig(context, cliOptions); |     const {plugins, options} = await getConfig(context, cliOptions); | ||||||
|  |     options.originalRepositoryURL = options.repositoryUrl; | ||||||
|     context.options = options; |     context.options = options; | ||||||
|     try { |     try { | ||||||
|       const result = await run(context, plugins); |       const result = await run(context, plugins); | ||||||
|  | |||||||
| @ -90,6 +90,7 @@ test('Plugins are called with expected values', async (t) => { | |||||||
|   const config = { |   const config = { | ||||||
|     branches: [{name: 'master'}, {name: 'next'}], |     branches: [{name: 'master'}, {name: 'next'}], | ||||||
|     repositoryUrl, |     repositoryUrl, | ||||||
|  |     originalRepositoryURL: repositoryUrl, | ||||||
|     globalOpt: 'global', |     globalOpt: 'global', | ||||||
|     tagFormat: `v\${version}`, |     tagFormat: `v\${version}`, | ||||||
|   }; |   }; | ||||||
| @ -927,7 +928,12 @@ test('Log all "verifyConditions" errors', async (t) => { | |||||||
|   const error2 = new SemanticReleaseError('error 2', 'ERR2'); |   const error2 = new SemanticReleaseError('error 2', 'ERR2'); | ||||||
|   const error3 = new SemanticReleaseError('error 3', 'ERR3'); |   const error3 = new SemanticReleaseError('error 3', 'ERR3'); | ||||||
|   const fail = stub().resolves(); |   const fail = stub().resolves(); | ||||||
|   const config = {branches: [{name: 'master'}], repositoryUrl, tagFormat: `v\${version}`}; |   const config = { | ||||||
|  |     branches: [{name: 'master'}], | ||||||
|  |     repositoryUrl, | ||||||
|  |     originalRepositoryURL: repositoryUrl, | ||||||
|  |     tagFormat: `v\${version}`, | ||||||
|  |   }; | ||||||
|   const options = { |   const options = { | ||||||
|     ...config, |     ...config, | ||||||
|     plugins: false, |     plugins: false, | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user