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

Support using Proxy for ServiceNow URL #97

Closed
mhackethal opened this issue Oct 22, 2019 · 4 comments · Fixed by #125
Closed

Support using Proxy for ServiceNow URL #97

mhackethal opened this issue Oct 22, 2019 · 4 comments · Fixed by #125
Assignees

Comments

@mhackethal
Copy link

Hi ,

i just want to query the Status i see an already open "fork" #7 for my Question but it was never commited since ~ 1 year.
Any Chance this feature will be inserted ?
I have to provide a Proxy with credentials or at least Parameter DefaultCredentials to Invoke-RestMethod.

Cheers

Markus

@Rick-2CA
Copy link
Member

Rick-2CA commented Nov 6, 2019

I haven't had to deal with a proxy before. Is this just a requirement to support Proxy on Invoke-RestMethod or is there more to it that we'd have to solve for?

@Rick-2CA Rick-2CA self-assigned this Nov 6, 2019
@mhackethal
Copy link
Author

you need to use it on every Invoke-RestMethod or Invoke-Webrequest call. There are some parameters that need to be provided.

  • ProxyURL
  • either UseDefaultCredentials or provide specific credentials like username/Password that is required to authenticate for the proxy.

I think its a bit tricky to implement this if you dont have any environment using a proxy. I just fixed it for me personal use with a dirty hack ;) but its nothing i can push here.

@Rick-2CA
Copy link
Member

Rick-2CA commented Nov 7, 2019

Would probably be best to force that functionality into Set-ServiceNowAuth. From there it wouldn't be hard to inject it into the right spots for the invoke calls. Trying to support it on a per command basis as a parameter would be more work and add even more parameters to each command. I'm hesitant to do more of that.

I'll leave this open and see what can be done. I've needed to set some time aside for another round of updates anyway. Busy couple months coming up though. Thanks for the request.

@Random206
Copy link

I would love for this feature to be added please.

@gdbarron gdbarron assigned gdbarron and unassigned Rick-2CA Apr 13, 2021
@gdbarron gdbarron mentioned this issue Apr 13, 2021
@gdbarron gdbarron linked a pull request Apr 13, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants