Uploaded image for project: 'Apache NiFi'
  1. Apache NiFi
  2. NIFI-10597

Change 'App Config File' property to external resource in Box processors

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 1.19.0
    • Extensions
    • None

    Description

      The Box app configuration can be specified as JSON file for Box processors, see

      "app-config-file" in JsonConfigBasedBoxClientService.

      Google processors use GCPCredentialsControllerService which includes a similar property for the configuration file in JSON format: "service-account-json-file".

      The "service-account-json-file" property uses 

      .identifiesExternalResource(ResourceCardinality.SINGLE, ResourceType.FILE)

      in its PropertyDescriptor,

      while "app-config-file" in Box case uses

      .addValidator(StandardValidators.FILE_EXISTS_VALIDATOR)

      It'd be useful to replace the StandardValidators.FILE_EXISTS_VALIDATOR usage with 
      identifiesExternalResource in the Box case. It contains the same check for file existence and also checks file accessbility, does not allow to specify directory instead of file.

      Attachments

        Issue Links

          Activity

            People

              kzsihovszki Zsihovszki Krisztina
              kzsihovszki Zsihovszki Krisztina
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 0.5h
                  0.5h