unifi-cli 0.2.2__tar.gz → 0.2.3__tar.gz

{unifi_cli-0.2.2 → unifi_cli-0.2.3}/.github/workflows/release.yml

@@ -332,7 +332,7 @@ jobs:
 
       - name: Update Homebrew formula
         env:
-          GH_TOKEN: ${{ secrets.HOMEBREW_TAP_TOKEN }}
+          HOMEBREW_TAP_DEPLOY_KEY: ${{ secrets.HOMEBREW_TAP_DEPLOY_KEY }}
         run: |
           VERSION="${{ github.ref_name }}"
           VERSION_NUM="${VERSION#v}"
@@ -381,8 +381,15 @@ jobs:
           sed -i "s/SHA_AARCH64_UNKNOWN_LINUX_GNU/${{ steps.hashes.outputs.aarch64_unknown_linux_gnu }}/g" /tmp/unifi-cli.rb
           sed -i "s/SHA_X86_64_UNKNOWN_LINUX_GNU/${{ steps.hashes.outputs.x86_64_unknown_linux_gnu }}/g" /tmp/unifi-cli.rb
 
-          # Clone tap repo, update formula, push
-          git clone https://x-access-token:${GH_TOKEN}@github.com/rvben/homebrew-tap.git /tmp/tap
+          # Authenticate to the tap with a write deploy key scoped to
+          # rvben/homebrew-tap. The key does not expire, so cross-repo pushes
+          # keep working without periodic token rotation.
+          mkdir -p ~/.ssh
+          echo "$HOMEBREW_TAP_DEPLOY_KEY" > ~/.ssh/id_ed25519
+          chmod 600 ~/.ssh/id_ed25519
+          ssh-keyscan -t ed25519 github.com >> ~/.ssh/known_hosts 2>/dev/null
+
+          git clone git@github.com:rvben/homebrew-tap.git /tmp/tap
           mkdir -p /tmp/tap/Formula
           cp /tmp/unifi-cli.rb /tmp/tap/Formula/unifi-cli.rb
           cd /tmp/tap

unifi_cli-0.2.3/.gitignore

@@ -0,0 +1,2 @@
+/target
+.claude/investigations/

{unifi_cli-0.2.2 → unifi_cli-0.2.3}/CHANGELOG.md

@@ -4,6 +4,13 @@ All notable changes to this project will be documented in this file.
 
 
 
+
+## [0.2.3](https://github.com/rvben/unifi-cli/compare/v0.2.2...v0.2.3) - 2026-07-07
+
+### Fixed
+
+- **events**: fall back to rest/alarm when stat/event is 404 on UniFi OS 9+ ([ac8a69c](https://github.com/rvben/unifi-cli/commit/ac8a69c743c5eab05def7bcd67e08e5b060275ea))
+
 ## [0.2.2](https://github.com/rvben/unifi-cli/compare/v0.2.1...v0.2.2) - 2026-06-11
 
 ### Added

{unifi_cli-0.2.2 → unifi_cli-0.2.3}/Cargo.lock

@@ -2786,7 +2786,7 @@ checksum = "ebc1c04c71510c7f702b52b7c350734c9ff1295c464a03335b00bb84fc54f853"
 
 [[package]]
 name = "unifi-cli"
-version = "0.2.2"
+version = "0.2.3"
 dependencies = [
  "clap",
  "clap_complete",

{unifi_cli-0.2.2 → unifi_cli-0.2.3}/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "unifi-cli"
-version = "0.2.2"
+version = "0.2.3"
 edition = "2024"
 rust-version = "1.90"
 description = "CLI for UniFi Network controller"

{unifi_cli-0.2.2 → unifi_cli-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: unifi-cli
-Version: 0.2.2
+Version: 0.2.3
 Classifier: Development Status :: 3 - Alpha
 Classifier: Environment :: Console
 Classifier: Intended Audience :: System Administrators

{unifi_cli-0.2.2 → unifi_cli-0.2.3}/src/api/client.rs

@@ -367,11 +367,30 @@ impl UnifiClient {
     }
 
     // Events
+    //
+    // Legacy `stat/event` was removed in UniFi Network 9+ (UniFi OS) and now
+    // returns api.err.NotFound (404). On those controllers the surviving REST
+    // surface for notable events is `rest/alarm`, whose records share this
+    // `Event` shape, so fall back to it. (The full live event stream on newer
+    // controllers is only exposed over the events WebSocket, which this REST
+    // client does not consume.)
     pub async fn list_events(&self, limit: usize) -> Result<Vec<Event>, ApiError> {
-        let events: Vec<Event> = self
-            .get_legacy(&format!("/stat/event?_limit={limit}"))
-            .await?;
-        Ok(events)
+        match self
+            .get_legacy::<Event>(&format!("/stat/event?_limit={limit}"))
+            .await
+        {
+            Ok(events) => Ok(events),
+            Err(ApiError::NotFound(_)) => {
+                let mut alarms: Vec<Event> = self.get_legacy("/rest/alarm").await?;
+                // `rest/alarm` is neither time-ordered nor limited server-side;
+                // present the most recent `limit` records to match the
+                // semantics `stat/event?_limit=` provided on older controllers.
+                alarms.sort_by_key(|e| std::cmp::Reverse(e.time));
+                alarms.truncate(limit);
+                Ok(alarms)
+            }
+            Err(e) => Err(e),
+        }
     }
 
     // Port table for a specific device

{unifi_cli-0.2.2 → unifi_cli-0.2.3}/tests/mock_server.rs

@@ -1706,6 +1706,80 @@ mod command_output {
             .unwrap_err();
         assert!(err.to_string().contains("Not found"));
     }
+
+    #[tokio::test]
+    async fn list_events_returns_stat_event_records() {
+        let server = MockServer::start().await;
+        Mock::given(method("GET"))
+            .and(path("/proxy/network/api/s/default/stat/event"))
+            .respond_with(ResponseTemplate::new(200).set_body_json(serde_json::json!({
+                "meta": {"rc": "ok"},
+                "data": [
+                    {"key": "EVT_AP_Connected", "msg": "AP connected", "subsystem": "wlan", "time": 200, "datetime": "2026-07-07T16:00:00Z"},
+                    {"key": "EVT_SW_LostContact", "msg": "Switch lost contact", "subsystem": "lan", "time": 100, "datetime": "2026-07-07T15:00:00Z"}
+                ]
+            })))
+            .mount(&server)
+            .await;
+
+        let client = mock_client(&server).await;
+        let events = client.list_events(10).await.unwrap();
+        assert_eq!(events.len(), 2);
+        assert_eq!(events[0].key.as_deref(), Some("EVT_AP_Connected"));
+    }
+
+    // UniFi Network 9+ (UniFi OS) removed the legacy stat/event route, which now
+    // returns api.err.NotFound (404). list_events must fall back to rest/alarm and
+    // return the most recent `limit` records.
+    #[tokio::test]
+    async fn list_events_falls_back_to_alarms_on_404() {
+        let server = MockServer::start().await;
+        Mock::given(method("GET"))
+            .and(path("/proxy/network/api/s/default/stat/event"))
+            .respond_with(ResponseTemplate::new(404).set_body_json(serde_json::json!({
+                "meta": {"rc": "error", "msg": "api.err.NotFound"},
+                "data": []
+            })))
+            .expect(1)
+            .mount(&server)
+            .await;
+        Mock::given(method("GET"))
+            .and(path("/proxy/network/api/s/default/rest/alarm"))
+            .respond_with(ResponseTemplate::new(200).set_body_json(serde_json::json!({
+                "meta": {"rc": "ok"},
+                "data": [
+                    {"key": "EVT_GW_Older", "msg": "older", "time": 100, "datetime": "a"},
+                    {"key": "EVT_GW_Newest", "msg": "newest", "time": 300, "datetime": "c"},
+                    {"key": "EVT_GW_Middle", "msg": "middle", "time": 200, "datetime": "b"}
+                ]
+            })))
+            .expect(1)
+            .mount(&server)
+            .await;
+
+        let client = mock_client(&server).await;
+        let events = client.list_events(2).await.unwrap();
+        // Most-recent-first, truncated to the requested limit.
+        assert_eq!(events.len(), 2);
+        assert_eq!(events[0].msg.as_deref(), Some("newest"));
+        assert_eq!(events[1].msg.as_deref(), Some("middle"));
+    }
+
+    #[tokio::test]
+    async fn list_events_propagates_non_404_errors() {
+        let server = MockServer::start().await;
+        Mock::given(method("GET"))
+            .and(path("/proxy/network/api/s/default/stat/event"))
+            .respond_with(ResponseTemplate::new(500).set_body_json(serde_json::json!({
+                "meta": {"rc": "error", "msg": "api.err.ServerError"},
+                "data": []
+            })))
+            .mount(&server)
+            .await;
+
+        let client = mock_client(&server).await;
+        assert!(client.list_events(10).await.is_err());
+    }
 }
 
 // --- Client construction tests ---

unifi_cli-0.2.2/.gitignore

@@ -1 +0,0 @@
-/target