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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        387d 21h 31m 1 Guillaume Nodet 15/Mar/12 17:01
        Resolved Resolved Closed Closed
        1096d 13h 59m 1 Carsten Ziegeler 17/Mar/15 07:00
        Carsten Ziegeler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Guillaume Nodet made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Guillaume Nodet [ gnt ]
        Fix Version/s fileinstall-3.1.12 [ 12316135 ]
        Resolution Fixed [ 1 ]
        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.
        Carl Hall made changes -
        Priority Major [ 3 ] Critical [ 2 ]
        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.
        Carl Hall made changes -
        Field Original Value New Value
        Attachment felix-2852.diff [ 12472917 ]
        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 -

        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.
        Carl Hall created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development