groovy
  1. groovy
  2. GROOVY-3921

Add connection configuration options to URL.getText(), URL.newInputStream() and URL.newReader()

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.8.1
    • Component/s: groovy-jdk
    • Labels:
      None
    • Patch Submitted:
      Yes
    • Number of attachments :
      1

      Description

      There was a question on StackOverflow about how to set a timeout for URL.text

      http://stackoverflow.com/questions/1839435/setting-timeout-for-new-url-text-in-groovy-grails

      I have attached a patch which allows you to do:

      def s = new URL( 'http://www.google.com' ).getText( 10 )

      (which will give you a 10 millisecond timeout on both the connection and read – and return null if this timeout fires)

      No Tests Supplied

      I have no idea how you would test this in a non-contrived way, I have looked around for a URL.text test and it seems that I would need to mock the URL class, but I can't think how to do the timeout testing with this object

      Hope it's ok...

        Activity

        CÚdric Champeau made changes -
        Field Original Value New Value
        Assignee Cedric Champeau [ melix ]
        CÚdric Champeau made changes -
        Summary Add a Timeout to URL.text Add connection configuration options to URL.getText(), URL.newInputStream() and URL.newReader()
        CÚdric Champeau made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.8.1 [ 17223 ]
        Resolution Fixed [ 1 ]
        Paul King made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            CÚdric Champeau
            Reporter:
            Tim Yates
          • Votes:
            3 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: