Uploaded image for project: 'OpenICF'
  1. OpenICF
  2. OPENICF-789

Missing check for headerUid if it exists in csv file data

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: csvfile-connector-1.5.3.0
    • Fix Version/s: csvfile-connector-1.5.4.0
    • Component/s: CSV File Connector
    • Labels:
    • Environment:
      Linux Mint 16.04, OpenIDM version "6.0.0-SNAPSHOT" (revision: 60671c1), CSV File Connector "1.5.3.0-SNAPSHOT" (Implementation-Build: 107)

      Description

      When started OpenIDM with CSV File connector so it should check if headerUid is existing at csvFile.

      https://stash.forgerock.org/projects/OPENICF/repos/csvfile-connector/browse/src/main/java/org/forgerock/openicf/csvfile/CSVFileConfiguration.java#137

       

      Steps for reproduce:

      • For the sample sync-with-csv
      1. Change provisioner.openicf-csvfile.json >> Set up headerUid at section configurationProperties on not exists value (e.g. "myNotExistsId")
      2. Running the sample
        ./startup.sh -p samples/sync-with-csv/
        

      Actual result: No one error/warning displayed.

      Executing ./startup.sh...
      Using OPENIDM_HOME:   /home/oem/apps/openidm
      Using PROJECT_HOME:   /home/oem/apps/openidm/samples/sync-with-csv
      Using OPENIDM_OPTS:   -Xmx1024m -Xms1024m
      Using LOGGING_CONFIG: -Djava.util.logging.config.file=/home/oem/apps/openidm/samples/sync-with-csv/conf/logging.properties
      -> OpenIDM version "6.0.0-SNAPSHOT" (revision: 60671c1) jenkins-OpenIDM - postcommit-3403 origin/master
      OpenIDM ready
      

      Expected result: Some error/warning should be displayed.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ben.apple Ben Apple
                Reporter:
                miroslav.meca Miroslav Meca
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: