From 804fd3de8e32c87e0c8a73d5db70360ae216a11b Mon Sep 17 00:00:00 2001 From: Lucien Greathouse Date: Fri, 22 Jan 2021 13:41:55 -0700 Subject: [PATCH] Remove Requester header to handle API change --- CHANGELOG.md | 1 + src/cli/upload.rs | 7 +++++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 59b8b038..f1a5b4ff 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,6 +1,7 @@ # Rojo Changelog ## Unreleased Changes +* Fixed `rojo upload` requests being rejected by Roblox ## [6.0.0](https://github.com/rojo-rbx/rojo/releases/tag/v6.0.0) (January 16, 2021) * Improved server error messages diff --git a/src/cli/upload.rs b/src/cli/upload.rs index 49e367d1..20d2d28f 100644 --- a/src/cli/upload.rs +++ b/src/cli/upload.rs @@ -53,13 +53,16 @@ pub fn upload(options: UploadCommand) -> Result<(), anyhow::Error> { .post(&url) .header(COOKIE, format!(".ROBLOSECURITY={}", &cookie)) .header(USER_AGENT, "Roblox/WinInet") - .header("Requester", "Client") .header(CONTENT_TYPE, "application/xml") .header(ACCEPT, "application/json") .body(buffer) .send()?; - if !response.status().is_success() { + let status = response.status(); + + if !status.is_success() { + log::error!("Error uploading, status: {}", status); + return Err(Error::RobloxApi { body: response.text()?, }