filebrowser

ix-dev/community/filebrowser/app.yaml

@@ -1,3 +1,5 @@
+annotations:
+  min_scale_version: '25.04'
 app_version: v2.32.0
 capabilities: []
 categories:
@@ -33,4 +35,4 @@ sources:
 - https://hub.docker.com/r/filebrowser/filebrowser
 title: File Browser
 train: community
-version: 1.2.8
+version: 1.3.0

ix-dev/community/filebrowser/migrations.yaml

@@ -0,0 +1,6 @@
+migrations:
+  - file: ip_port_migration
+    # from:
+    #   max_version: "FIXME:"
+    target:
+      min_version: 1.3.0

ix-dev/community/filebrowser/migrations/ip_port_migration

@@ -0,0 +1,23 @@
+#!/usr/bin/python3
+
+import os
+import sys
+import yaml
+
+
+def migrate(values):
+    values["network"]["web_port"] = {
+        "port_number": values["network"]["web_port"],
+        "bind_mode": "published",
+        "host_ips": [],
+    }
+    return values
+
+
+if __name__ == "__main__":
+    if len(sys.argv) != 2:
+        exit(1)
+
+    if os.path.exists(sys.argv[1]):
+        with open(sys.argv[1], "r") as f:
+            print(yaml.dump(migrate(yaml.safe_load(f.read()))))

ix-dev/community/filebrowser/questions.yaml

@@ -72,13 +72,51 @@ questions:
       attrs:
         - variable: web_port
           label: WebUI Port
-          description: The port for File Browser WebUI
           schema:
-            type: int
-            default: 30051
-            required: true
-            $ref:
-              - definitions/port
+            type: dict
+            attrs:
+              - variable: bind_mode
+                label: Port Bind Mode
+                description: |
+                  The port bind mode.</br>
+                  - Publish: The port will be published on the host for external access.</br>
+                  - Expose: The port will be exposed for inter-container communication.</br>
+                  - None: The port will not be exposed or published.</br>
+                  Note: If the Dockerfile defines an EXPOSE directive,
+                  the port will still be exposed for inter-container communication regardless of this setting.
+                schema:
+                  type: string
+                  default: "published"
+                  enum:
+                    - value: "published"
+                      description: Publish port on the host for external access
+                    - value: "exposed"
+                      description: Expose port for inter-container communication
+                    - value: ""
+                      description: None
+              - variable: port_number
+                label: Port Number
+                schema:
+                  type: int
+                  show_if: [["bind_mode", "!=", ""]]
+                  default: 30051
+                  required: true
+                  $ref:
+                    - definitions/port
+              - variable: host_ips
+                label: Host IPs
+                description: IPs on the host to bind this port
+                schema:
+                  type: list
+                  default: []
+                  items:
+                    - variable: host_ip
+                      label: Host IP
+                      schema:
+                        type: string
+                        required: true
+                        $ref:
+                          - definitions/node_bind_ip
         - variable: host_network
           label: Host Network
           description: |

ix-dev/community/filebrowser/templates/docker-compose.yaml

@@ -25,7 +25,7 @@
 {% set cmd = namespace(x=[
   "--config", "%s/filebrowser.json"|format(values.consts.config_path),
   "--database", "%s/filebrowser.db"|format(values.consts.config_path),
-  "--port", "%s"|format(values.network.web_port),
+  "--port", "%s"|format(values.network.web_port.port_number),
   "--address", "0.0.0.0",
   "--root", values.consts.data_path,
 ]) %}
@@ -41,10 +41,10 @@
 
 {% do c1.set_command(cmd.x) %}
 
-{% do c1.healthcheck.set_test("curl", {"port": values.network.web_port, "path": "/health", "scheme": proto}) %}
+{% do c1.healthcheck.set_test("curl", {"port": values.network.web_port.port_number, "path": "/health", "scheme": proto}) %}
 {% do c1.environment.add_user_envs(values.filebrowser.additional_envs) %}
 
-{% do c1.ports.add_port(values.network.web_port, values.network.web_port) %}
+{% do c1.add_port(values.network.web_port) %}
 
 {% do c1.add_storage(values.consts.config_path, values.storage.config) %}
 {% do init.add_storage(values.consts.config_path, values.storage.config) %}
@@ -63,6 +63,6 @@
   {% do init.depends.add_dependency(values.consts.perms_container_name, "service_completed_successfully") %}
 {% endif %}
 
-{% do tpl.portals.add_portal({"port": values.network.web_port, "scheme": proto}) %}
+{% do tpl.portals.add_portal({"port": values.network.web_port.port_number, "scheme": proto}) %}
 
 {{ tpl.render() | tojson }}

ix-dev/community/filebrowser/templates/test_values/basic-values.yaml

@@ -8,7 +8,9 @@ filebrowser:
 network:
   host_network: false
   certificate_id:
-  web_port: 8080
+  web_port:
+    bind_mode: published
+    port_number: 8080
 
 run_as:
   user: 568

ix-dev/community/filebrowser/templates/test_values/https-values.yaml

@@ -8,7 +8,9 @@ filebrowser:
 network:
   host_network: false
   certificate_id: "1"
-  web_port: 8080
+  web_port:
+    bind_mode: published
+    port_number: 8080
 
 run_as:
   user: 568