Commons DbUtils
  1. Commons DbUtils
  2. DBUTILS-81

DbUtils.loadDriver() uses Class.forName()

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      The Class.forName() statement should be avoided due to potential OSGi issues - commons components are OSGi bundles!
      The ClassLoader should be used instead to load classes and add a new method to pass custom ClassLoader.

      1. DBUTILS-81.patch
        2 kB
        Simone Tripodi

        Activity

        Simone Tripodi created issue -
        Hide
        Sebb added a comment -

        Have you a suggested patch for this?

        Show
        Sebb added a comment - Have you a suggested patch for this?
        Hide
        Simone Tripodi added a comment -

        I can provide one, it worths anyway having DBUTILS-80 fixed first

        Show
        Simone Tripodi added a comment - I can provide one, it worths anyway having DBUTILS-80 fixed first
        Hide
        Simone Tripodi added a comment -

        Please take a loot at the proposed patch, if it works for you I can commit.
        Looking forward your feedback

        Show
        Simone Tripodi added a comment - Please take a loot at the proposed patch, if it works for you I can commit. Looking forward your feedback
        Simone Tripodi made changes -
        Field Original Value New Value
        Attachment DBUTILS-81.patch [ 12496100 ]
        Hide
        Simone Tripodi added a comment -

        Got a feedback from Neil Bartlett (one of the OSGi gurus) on Twitter, going to commit the patch and asking for feedbacks on ML

        Show
        Simone Tripodi added a comment - Got a feedback from Neil Bartlett (one of the OSGi gurus) on Twitter , going to commit the patch and asking for feedbacks on ML
        Simone Tripodi made changes -
        Hide
        Simone Tripodi added a comment -

        fixed on r1175267

        Show
        Simone Tripodi added a comment - fixed on r1175267
        Simone Tripodi made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Simone Tripodi [ simone.tripodi ]
        Resolution Fixed [ 1 ]
        Henri Yandell made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2d 13h 5m 1 Simone Tripodi 24/Sep/11 22:28
        Resolved Resolved Closed Closed
        46d 20h 39m 1 Henri Yandell 10/Nov/11 18:07

          People

          • Assignee:
            Simone Tripodi
            Reporter:
            Simone Tripodi
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development