fix: Add user-agent to newm-admin requests

Author: AndrewWestberg

.github/workflows/newm-admin.yml

@@ -213,16 +213,16 @@ jobs:
           VERSION=$(sed -n 's/^version = "\(.*\)"/\1/p' Cargo.toml | head -n1)
           echo "VERSION=${VERSION}" >> $GITHUB_OUTPUT
 
-      - name: Initialize and Build MSI
+      - name: Build MSI
+        shell: pwsh
         run: |
-          cargo wix init --force
-          cargo wix --nocapture --target x86_64-pc-windows-msvc
+          ./packaging/windows/build-msi.ps1 -Version "${{ steps.version.outputs.VERSION }}" -Target "x86_64-pc-windows-msvc"
 
       - name: Upload MSI
         uses: actions/upload-artifact@v4
         with:
           name: NEWM-Admin-${{ steps.version.outputs.VERSION }}-windows.msi
-          path: newm-admin/target/wix/*.msi
+          path: newm-admin/target/x86_64-pc-windows-msvc/release/NEWM-Admin-${{ steps.version.outputs.VERSION }}-windows.msi
 
   # Release job uploads all packaged artifacts to GitHub Releases
   release:

newm-admin/Cargo.lock

@@ -1,7 +1,10 @@
 [package]
 name = "newm-admin"
-version = "0.1.0"
+version = "0.1.1"
 edition = "2024"
+authors = ["Andrew Westberg <awestberg@newm.io>"]
+license = "Apache-2.0"
+license-file = "../LICENSE"
 
 [dependencies]
 # GUI Framework

newm-admin/Cargo.toml

@@ -6,6 +6,7 @@
 //! Uses async reqwest with async-compat for Tokio compatibility
 //! within GPUI's async executor.
 
+use crate::http_client;
 use reqwest::Client;
 use serde::{Deserialize, Serialize};
 use std::fmt;
@@ -133,7 +134,7 @@ impl AuthClient {
     /// Create a new authentication client
     pub fn new() -> Self {
         Self {
-            client: Client::new(),
+            client: http_client::new_client(),
         }
     }
 

newm-admin/src/auth.rs

@@ -6,6 +6,7 @@ use async_compat::Compat;
 use reqwest::Client;
 use serde::{Deserialize, Serialize};
 
+use crate::http_client;
 use crate::session::{Session, SessionError};
 
 /// Request to add royalties to a song
@@ -81,7 +82,7 @@ impl EarningsClient {
     /// Create a new earnings client
     pub fn new() -> Self {
         Self {
-            client: Client::new(),
+            client: http_client::new_client(),
         }
     }
 

newm-admin/src/earnings.rs

@@ -0,0 +1,23 @@
+//! Shared HTTP client configuration for NEWM Admin.
+//!
+//! Studio traffic is fronted by an AWS load balancer that rejects requests without a non-empty
+//! `User-Agent` header (returns an HTML 403). To keep behavior consistent across environments,
+//! we configure a default `User-Agent` for all outbound reqwest requests.
+
+use reqwest::{Client, header};
+
+const USER_AGENT: &str = concat!("newm-admin/", env!("CARGO_PKG_VERSION"));
+
+pub fn new_client() -> Client {
+    Client::builder()
+        .default_headers({
+            let mut headers = header::HeaderMap::new();
+            headers.insert(
+                header::USER_AGENT,
+                header::HeaderValue::from_static(USER_AGENT),
+            );
+            headers
+        })
+        .build()
+        .expect("Failed to build reqwest client")
+}

newm-admin/src/http_client.rs

@@ -3,6 +3,7 @@ mod auth;
 mod colors;
 mod csv_import;
 mod earnings;
+mod http_client;
 mod jwt;
 mod session;
 mod toast;

newm-admin/src/main.rs

@@ -5,10 +5,10 @@
 
 use async_compat::Compat;
 use gpui::*;
-use reqwest::Client;
 use std::sync::{Arc, Mutex};
 
 use crate::auth::{Environment, LoginResponse};
+use crate::http_client;
 use crate::jwt;
 
 /// Session state containing authentication tokens
@@ -82,7 +82,7 @@ impl Session {
         tracing::info!("Access token expires soon, refreshing...");
 
         // Need to refresh - use the refresh token
-        let client = Client::new();
+        let client = http_client::new_client();
         let url = self.environment.refresh_url();
 
         let response = Compat::new(async {