Skip to content
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

Open
cjwolford opened this issue Jun 15, 2016 · 4 comments
Open

Missing P Drive #6

cjwolford opened this issue Jun 15, 2016 · 4 comments

Comments

@cjwolford
Copy link

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"

@RamblingCookieMonster
Copy link
Owner

Thanks for the heads up, missed a few similar changes in this, will have to read through that more carefully!

@cjwolford
Copy link
Author

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.

@RamblingCookieMonster
Copy link
Owner

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!

@bushe
Copy link
Contributor

bushe commented Jun 15, 2016

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants