Uploaded image for project: 'JBRULES'
  1. JBRULES
  2. JBRULES-3667

UrlResource getLastModified() method does not use authentication

This issue belongs to an archived project. You can view it, but you can't modify it. Learn more

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Major
    • None
    • 5.4.0.Final, 5.5.0.Beta1
    • drools-core
    • None
    • Hide

      I recreated this using Guvnor:
      Create a guvnor default installation that requires authentication.
      Create a client that uses a change-set that uses the aforementioned resource.

      The behavior in guvnor is that ResourceChangeScannerImpl will unsubscribe the stream when the first "check" call is made by default in 1 minute increments.

      Show
      I recreated this using Guvnor: Create a guvnor default installation that requires authentication. Create a client that uses a change-set that uses the aforementioned resource. The behavior in guvnor is that ResourceChangeScannerImpl will unsubscribe the stream when the first "check" call is made by default in 1 minute increments.

    Description

      Drools supports BASIC authentication when fetching packages via a UrlResource. The implementation is complete for the grabStream() method but is not present for the getLastModified() method.

      The latter method is used by clients to determine if packages have changed over time. For example, this is use by Guvnor when the KnowledgeAgent (via the ResourceChangeScannerImpl) needs to determine if a remote URL based package has been changed.

      The defect that happens is swallowed, in that the result of the getLastModified() call returns a zero because of the redirect to whatever authentication is required. So no exception or anything is produced. The client just gets a zero back.

      The core should be consistent on when it uses authentication and should use it everywhere that URL connections are used.

      Attachments

        Activity

          People

            mproctor@redhat.com Mark Proctor
            matthewdfleming Matthew Fleming (Inactive)
            Archiver:
            rhn-support-ceverson Clark Everson

            Dates

              Created:
              Updated:
              Archived:

              PagerDuty