-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing P Drive #6
Comments
Thanks for the heads up, missed a few similar changes in this, will have to read through that more carefully! |
I'm also seeing it trying to access $Uri="https://pwmanager.corp.athenahealth.com/SecretServer/webservices/SSWebservice.asmx" It certainly has a lot of functionality once its configured. |
Alrighty! This is a bit far down my list of things to do (don't work with Secret Server anymore, unfortunately) - would definitely be up for a pull request fixing these if you're up for it! Alternatively, we could roll back to before the PR that introduced these and see if @bushe would be interested in adding his changes back - Ryan, let me know if you're up for that. Would probably prefer to keep the initial file structure, it's slightly slower but a bit more readable and manageable with one file per function. Cheers! |
I will work on cleaning up some of the hard coded stuff. Yeah, I can keep the initial file structure for any additional pull requests. EDIT: I wasn't aware that you pulled in my restructuring. Let me see what I can do. |
When trying to set this up I was given errors that the P drive didn't exist.
In SecretServer.psm1 on line 1462 the following command is trying to look for a mapped drive.
Import-Clixml -Path "P:\Scripts\SecretServer\SecretServer\SecretServer_$($env:USERNAME).xml"
The text was updated successfully, but these errors were encountered: