Felix
  1. Felix
  2. FELIX-2852

Infinite reloading of config file ending with .config

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: fileinstall-3.1.10
    • Fix Version/s: fileinstall-3.2.0
    • Component/s: File Install
    • Labels:
      None

      Description

      When a config file ending with .config is written back to the file system, lastModified is updated even if the file has not changed since it was last written/read. Config files ending with .cfg are not affected by this as lastModified is not updated if the file has not changed.

      Workaround: apply the patch from FELIX-2843 and set felix.fileinstall.disableConfigSave=false

        Activity

        Hide
        Carl Hall added a comment -

        I have a preliminary patch put together to update the .config file's checksum in the scanner but have hardcoded how the scanner is retrieved. I will continue to refine the patch and attach it once I am satisfied with it.

        Show
        Carl Hall added a comment - I have a preliminary patch put together to update the .config file's checksum in the scanner but have hardcoded how the scanner is retrieved. I will continue to refine the patch and attach it once I am satisfied with it.
        Hide
        Carl Hall added a comment -

        Attached a patch to manually update a file's checksum after writing a .config file. The checksum is only updated in scanners that have prior knowledge of the file.

        Show
        Carl Hall added a comment - Attached a patch to manually update a file's checksum after writing a .config file. The checksum is only updated in scanners that have prior knowledge of the file.
        Hide
        Carl Hall added a comment -

        Bumping up to "critical" as infinite loops can get painful. Not a blocker as there is a workaround.

        Show
        Carl Hall added a comment - Bumping up to "critical" as infinite loops can get painful. Not a blocker as there is a workaround.
        Hide
        Johan Ström added a comment - - edited

        ~1 year later; is there any update on this bug? I still have this problem running 3.1.10 (as shipped with Apache Karaf 2.2.5). As 3.1.10 is the latest version released, and the workaround doesn't work on the released version, I'd say this is a problem. Patching trunk manually isn't too production-friendly.

        Show
        Johan Ström added a comment - - edited ~1 year later; is there any update on this bug? I still have this problem running 3.1.10 (as shipped with Apache Karaf 2.2.5). As 3.1.10 is the latest version released, and the workaround doesn't work on the released version, I'd say this is a problem. Patching trunk manually isn't too production-friendly.

          People

          • Assignee:
            Guillaume Nodet
            Reporter:
            Carl Hall
          • Votes:
            3 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development