84 Commits (19bd28cd72a7ade96ab21986f568db4515bc9fa1)

Author	SHA1	Message	Date
Matt Dainty	19bd28cd72	Fix/add tests	7 years ago
Matt Dainty	2e4b00f59d	Refactor powershell provisioner ... Use the common elevated code instead.	7 years ago
Matt Dainty	412119c27e	Fix powershell provisioner tests	7 years ago
Matt Dainty	3b519acebc	Allow Powershell provisioner to use service accounts ... Fixes #6104	7 years ago
Megan Marsh	1d1729a5ff	fix powershell tests to use new command	7 years ago
Megan Marsh	a81d18cdab	explicitly use set-variable to work around a bug some users saw where the command invocation would vail	7 years ago
Megan Marsh	a0edaf6c46	Going to revert this change for now, becuase of potential issues that arise from calling Prepare() twice ... Revert "use statebag instead of SetSharedState for winRM password" This reverts commit b35acbd879.	8 years ago
Megan Marsh	b35acbd879	use statebag instead of SetSharedState for winRM password	8 years ago
Megan Marsh	ff6a039d5b	replace scrubconfig with packer.LogSecretFilter.Set ... filter winrm password from logs Add new root-level packer template option, sensitive-variables, to tell us what user variables to mark sensitive.	8 years ago
DanHam	485d565e32	Ensure comments are easily readable in a standard terminal	8 years ago
DanHam	e0bcba4913	Make the upload of env vars retryable in case of restarts	8 years ago
DanHam	b738f41efa	Remove tmp files created by provisioner/powershell tests	8 years ago
DanHam	b9ab2c8048	Remove script containing PowerShell inline commands from local tmp when done	8 years ago
DanHam	ccf687dac6	Fix tests	8 years ago
DanHam	5419375413	Change to CMD % style path is no longer required	8 years ago
DanHam	5b652316d9	Dollar's in env vars used in paths may cause problems for ssh ... * Dollars are interpreted by *nix shells so paths using env vars such as ${env:SYSTEMROOT} will cause issues	8 years ago
DanHam	4e7feea184	Allow users to specify the location that the env vars file is uploaded to ... Previously the location the file was uploaded to was set internally and used ${env:SYSTEMROOT}/Temp as the destination folder. By default, in order to inject the required environment variables, the file is 'dot sourced' by the 'execute_command' using the {{ .Vars }} variable. Unfortunately the inclusion of the dollar in the path caused issues for users connecting over ssh as the (typically bash) shell running the execute command would try and interpret the dollar sign. The change allows users to specify the location the file is uploaded to, thereby allowing the user to specify a custom 'execute_command' that escapes any dollar signs that could be present in the path. If not set the upload path now defaults to using C:/Windows/Temp as the upload folder.	8 years ago
Megan Marsh	3afb243f11	use build name to ensure that winrm password and other shared state is not overwritten if two builders need the password in the same packer run.	8 years ago
DanHam	ef4817d644	Fix vertical spacing	8 years ago
DanHam	34db6c4ab1	Remove temp debugging output	8 years ago
Megan Marsh	4e321b2dfa	don't need to use a receiver with this function	8 years ago
Megan Marsh	ce1ab1f021	fix winrm password	8 years ago
Megan Marsh	559719020c	use {{.WinRMPassword}} instead of some other weirdness	8 years ago
Megan Marsh	d689e6b4d3	allow users of AWS to use the dynamically-generated admin password which we use as the winRM password as an elevated password in the Powershell provisioner, as well as an environment variable in same provisoner.	8 years ago
Josh Soref	24b4c36fc9	spelling: function	8 years ago
Josh Soref	04d6bfc696	spelling: environment	8 years ago
Megan Marsh	f3a538db46	fix tests	8 years ago
Megan Marsh	3faf73b5f3	change backslashes to forward slashes in powershell provisioner; was breaking with cygwin	8 years ago
DanHam	2d830d5d43	Auto escape chars special to PowerShell in user supplied data	8 years ago
DanHam	aaf7102b9a	Tests for escape of chars special to PowerShell in user supplied data	8 years ago
DanHam	a7b118ed94	Fix tests post changes. Add test for upload func.	8 years ago
DanHam	478589abec	Extend upload and subsequent 'dot sourcing' of env vars to std PS command ... * Wrap funcs to flatten and upload env vars with new func prepareEnvVars. While the wrapped funcs could be combined, keeping them separate simplifies testing. * Configure/refactor std and elevated PS to use new funcs to prepare, upload and dot source env vars. * Dot sourcing the env vars in this way avoids the need to embed them directly in the command string. This avoids the need to escape the env vars to ensure the command string is correctly parsed. * Characters within the env vars that are special to PS (such as $'s and backticks) will still need to be escaped to allow them to be correctly interpreted by PS. * The std and elevated PS commands now inject env vars into the remote env via the same mechanism. This ensures consistent behaviour across the two command types. Fixes #5471	8 years ago
Megan Marsh	addedbb680	Revert "Merge pull request #5515 from DanHam/dot-source-env-vars" ... revert so we can use a branch that's had more recent work done This reverts commit e56849c605, reversing changes made to 6d14eb6ea4.	8 years ago
Megan Marsh	79fe900378	Revert "Merge pull request #5376 from DanHam/ps-escapes" ... Revert so that we can merge a different branch that's had more recent work instead This reverts commit ba518637d4, reversing changes made to e56849c605.	8 years ago
DanHam	7443adf2fd	Simpler escape of chars special to PowerShell in user supplied data	8 years ago
DanHam	68e13c90b1	Revert "Escape chars special to PowerShell in user supplied data" ... This reverts commit 53aefe744bcd74ee6ac866f764eafe9e7f507d80.	8 years ago
DanHam	b67c64fd66	Tests for escape of chars special to PowerShell in user supplied data	8 years ago
DanHam	f3c326bb3c	Escape chars special to PowerShell in user supplied data	8 years ago
DanHam	4b89fc1c00	Fix tests post changes. Add test for upload func.	8 years ago
DanHam	5949bc91c4	Extend upload and subsequent 'dot sourcing' of env vars to std PS command ... * Wrap funcs to flatten and upload env vars with new func prepareEnvVars. While the wrapped funcs could be combined, keeping them separate simplifies testing. * Configure/refactor std and elevated PS to use new funcs to prepare, upload and dot source env vars. * Dot sourcing the env vars in this way avoids the need to embed them directly in the command string. This avoids the need to escape the env vars to ensure the command string is correctly parsed. * Characters within the env vars that are special to PS (such as $'s and backticks) will still need to be escaped to allow them to be correctly interpreted by PS. * The std and elevated PS commands now inject env vars into the remote env via the same mechanism. This ensures consistent behaviour across the two command types. Fixes #5471	8 years ago
DanHam	fa5fd602aa	Fix tests post changes	9 years ago
DanHam	8214a12a2a	Move append of portion of command used to send elevated PS output to file	9 years ago
DanHam	2a060adbf8	Don't use -EncodedCommand with PS as progress stream always leaks to stderr ... * Setting $ProgressPreference to SilentlyContinue makes no difference when -EncodedCommand is used - any output to the progress stream still appears on stderr. * Delete file containing encode/decode functions since we no longer need them. * Fixes leak of output on progress streams for both normal and elevated commands. * Since we no longer base64 encode, ensure any characters special to XML are correctly escaped in the elevated command. This ensures correct parsing once the command is wrapped within the elevatedTemplates XML based Task Scheduler definition. Fixes #4322	9 years ago
DanHam	5374c6e367	Use system context env var to set path for elevated PS env vars file	9 years ago
Matthew Hooker	af1187d5e1	simplify a bit	9 years ago
Megan Marsh	b05c673a14	Update powershell provisioner test with new default	9 years ago
Megan Marsh	cbe1d7d854	simplify file upload for elevated powershell wrapper	9 years ago
Megan Marsh	92e70757bb	Move env vars into a file that we dot-source instead of trying to write them all to the command line	9 years ago
Megan Marsh	d458628529	exclude elevated_executed_command from config interpolation so it can be used correctly later	9 years ago
Megan Marsh	c6889ad009	clarify logging by changing 'shell' to 'powershell' inside of the powershell provisioner	9 years ago