Skip to content
This repository has been archived by the owner on Jun 18, 2024. It is now read-only.

Added support for a ignoreSSLIssues as a parameter on dynamic methods #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jamjon3
Copy link

@jamjon3 jamjon3 commented Sep 12, 2014

Allows the use of this as a parameter on dynamic methods using to the underlying http-builder support for it. Essentially it alternatively registers a scheme using the ignoreSSLIssues() function on the client rather than via SimpleHTTPBuilderSSLHelper.groovy

This addresses closed issue #14 where the http-builder was updated but the plugin had no handling for it (example in the comments)

…g to the underlying http-builder support for it. Registers a scheme using the ignoreSSLIssues() function on the client rather than via SimpleHTTPBuilderSSLHelper.groovy
@jamjon3
Copy link
Author

jamjon3 commented Sep 12, 2014

Works like this:

withRest(uri: "https://www.pcwebshop.co.uk/", ignoreSSLIssues: true) {
  get() { resp, reader ->
    /** Do some stuff here **/
  }
}

@@ -96,6 +96,7 @@ class RestGrailsPlugin {

private makeClient(Class klass, Map params, application) {
def client
def ignoreSSLIssues = (params.ignoreSSLIssues)?params.remove("ignoreSSLIssues"):false
Copy link
Author

Choose a reason for hiding this comment

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

Pulls out this parameter early

@bobbywarner bobbywarner force-pushed the master branch 2 times, most recently from a76ed23 to ee0e020 Compare January 14, 2016 19:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant