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

Invoke RubrikRESTCall #118

Merged
merged 2 commits into from
Oct 18, 2017
Merged

Invoke RubrikRESTCall #118

merged 2 commits into from
Oct 18, 2017

Conversation

MikeFal
Copy link
Contributor

@MikeFal MikeFal commented Oct 4, 2017

Adding function to allow users to make raw API calls to Rubrik. This enables functionality to call API endpoints not yet supported by the module.

Mike Fal added 2 commits October 4, 2017 09:21
-Update to Submit-Request to not rely on $resrouce object
-Added Invoke-RubrikRestCall
@@ -1,4 +1,4 @@
function Submit-Request($uri,$header,$method,$body)
function Submit-Request($uri,$header,$method = $($resources.Method) ,$body)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious why this change was required?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Due to the change made to line 15 for the Invoke-WebRequest. The original version was locked into whatever the $resources item was. I needed Submit-Request to not be dependent on the $resources collection (be more generic). However, to maintain the functionality I changed the method to default to $resources.Method .

Copy link
Member

@pmilano1 pmilano1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense.

@pmilano1 pmilano1 merged commit 5dfab5c into master Oct 18, 2017
@chriswahl chriswahl deleted the invokeRubrikRest branch November 4, 2017 00:26
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

Successfully merging this pull request may close these issues.

3 participants