From 1c2e910dea46e6dcab3c494f399869f389f52c64 Mon Sep 17 00:00:00 2001 From: Greg Burri Date: Fri, 9 May 2025 23:45:32 +0200 Subject: [PATCH] Replace mime type "application/ron" by "text/ron" and put it in the common crate --- Cargo.lock | 191 ++++++++++++++--------------------- backend/src/ron_extractor.rs | 4 +- backend/src/ron_utils.rs | 10 +- backend/src/services/mod.rs | 2 +- common/Cargo.toml | 1 + common/src/consts.rs | 6 ++ frontend/src/request.rs | 11 +- 7 files changed, 96 insertions(+), 129 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index c089e11..03635db 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -19,14 +19,14 @@ checksum = "512761e0bb2578dd7380c6baaa0f4ce03e84f95e960231d1dec8bf4d7d6e2627" [[package]] name = "ahash" -version = "0.8.11" +version = "0.8.12" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "e89da841a80418a9b391ebaea17f5c112ffaaa96f621d2c285b5174da76b9011" +checksum = "5a15f179cd60c4584b8a8c596927aadc462e27f2ca70c04e0071964a73ba7a75" dependencies = [ "cfg-if", "once_cell", "version_check", - "zerocopy 0.7.35", + "zerocopy", ] [[package]] @@ -499,9 +499,9 @@ dependencies = [ [[package]] name = "cc" -version = "1.2.21" +version = "1.2.22" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "8691782945451c1c383942c4874dbe63814f61cb57ef773cda2972682b7bb3c0" +checksum = "32db95edf998450acc7881c932f94cd9b05c87b4b2599e8bab064753da4acfd1" dependencies = [ "shlex", ] @@ -590,6 +590,7 @@ name = "common" version = "0.1.0" dependencies = [ "chrono", + "http 1.3.1", "ron", "serde", "strum", @@ -667,9 +668,9 @@ dependencies = [ [[package]] name = "crc" -version = "3.2.1" +version = "3.3.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "69e6e4d7b33a94f0991c26729976b10ebde1d34c3ee82408fb536164fa10d636" +checksum = "9710d3b3739c2e349eb44fe848ad0b7c8cb1e42bd87ee49371df2f7acaf3e675" dependencies = [ "crc-catalog", ] @@ -1184,9 +1185,9 @@ dependencies = [ [[package]] name = "getrandom" -version = "0.3.2" +version = "0.3.3" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "73fea8450eea4bac3940448fb7ae50d91f034f941199fcd9d909a5a07aa455f0" +checksum = "26145e563e54f2cadc477553f1ec5ee650b00862f0a58bcd12cbdc5f0ea2d2f4" dependencies = [ "cfg-if", "libc", @@ -1517,21 +1518,22 @@ dependencies = [ [[package]] name = "icu_collections" -version = "1.5.0" +version = "2.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "db2fa452206ebee18c4b5c2274dbf1de17008e874b4dc4f0aea9d01ca79e4526" +checksum = "200072f5d0e3614556f94a9930d5dc3e0662a652823904c3a75dc3b0af7fee47" dependencies = [ "displaydoc", + "potential_utf", "yoke", "zerofrom", "zerovec", ] [[package]] -name = "icu_locid" -version = "1.5.0" +name = "icu_locale_core" +version = "2.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "13acbb8371917fc971be86fc8057c41a64b521c184808a698c02acc242dbf637" +checksum = "0cde2700ccaed3872079a65fb1a78f6c0a36c91570f28755dda67bc8f7d9f00a" dependencies = [ "displaydoc", "litemap", @@ -1540,31 +1542,11 @@ dependencies = [ "zerovec", ] -[[package]] -name = "icu_locid_transform" -version = "1.5.0" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "01d11ac35de8e40fdeda00d9e1e9d92525f3f9d887cdd7aa81d727596788b54e" -dependencies = [ - "displaydoc", - "icu_locid", - "icu_locid_transform_data", - "icu_provider", - "tinystr", - "zerovec", -] - -[[package]] -name = "icu_locid_transform_data" -version = "1.5.1" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "7515e6d781098bf9f7205ab3fc7e9709d34554ae0b21ddbcb5febfa4bc7df11d" - [[package]] name = "icu_normalizer" -version = "1.5.0" +version = "2.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "19ce3e0da2ec68599d193c93d088142efd7f9c5d6fc9b803774855747dc6a84f" +checksum = "436880e8e18df4d7bbc06d58432329d6458cc84531f7ac5f024e93deadb37979" dependencies = [ "displaydoc", "icu_collections", @@ -1572,67 +1554,54 @@ dependencies = [ "icu_properties", "icu_provider", "smallvec", - "utf16_iter", - "utf8_iter", - "write16", "zerovec", ] [[package]] name = "icu_normalizer_data" -version = "1.5.1" +version = "2.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "c5e8338228bdc8ab83303f16b797e177953730f601a96c25d10cb3ab0daa0cb7" +checksum = "00210d6893afc98edb752b664b8890f0ef174c8adbb8d0be9710fa66fbbf72d3" [[package]] name = "icu_properties" -version = "1.5.1" +version = "2.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "93d6020766cfc6302c15dbbc9c8778c37e62c14427cb7f6e601d849e092aeef5" +checksum = "2549ca8c7241c82f59c80ba2a6f415d931c5b58d24fb8412caa1a1f02c49139a" dependencies = [ "displaydoc", "icu_collections", - "icu_locid_transform", + "icu_locale_core", "icu_properties_data", "icu_provider", - "tinystr", + "potential_utf", + "zerotrie", "zerovec", ] [[package]] name = "icu_properties_data" -version = "1.5.1" +version = "2.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "85fb8799753b75aee8d2a21d7c14d9f38921b54b3dbda10f5a3c7a7b82dba5e2" +checksum = "8197e866e47b68f8f7d95249e172903bec06004b18b2937f1095d40a0c57de04" [[package]] name = "icu_provider" -version = "1.5.0" +version = "2.0.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "6ed421c8a8ef78d3e2dbc98a973be2f3770cb42b606e3ab18d6237c4dfde68d9" +checksum = "03c80da27b5f4187909049ee2d72f276f0d9f99a42c306bd0131ecfe04d8e5af" dependencies = [ "displaydoc", - "icu_locid", - "icu_provider_macros", + "icu_locale_core", "stable_deref_trait", "tinystr", "writeable", "yoke", "zerofrom", + "zerotrie", "zerovec", ] -[[package]] -name = "icu_provider_macros" -version = "1.5.0" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "1ec89e9337638ecdc08744df490b221a7399bf8d164eb52a665454e60e075ad6" -dependencies = [ - "proc-macro2", - "quote", - "syn", -] - [[package]] name = "ident_case" version = "1.0.1" @@ -1652,9 +1621,9 @@ dependencies = [ [[package]] name = "idna_adapter" -version = "1.2.0" +version = "1.2.1" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "daca1df1c957320b2cf139ac61e7bd64fed304c5040df000a745aa1de3b4ef71" +checksum = "3acae9609540aa318d1bc588455225fb2085b9ed0c4f6bd0d9d5bcd86f1a0344" dependencies = [ "icu_normalizer", "icu_properties", @@ -1776,9 +1745,9 @@ checksum = "cd945864f07fe9f5371a27ad7b52a172b4b499999f1d97574c9fa68373937e12" [[package]] name = "litemap" -version = "0.7.5" +version = "0.8.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "23fb14cb19457329c82206317a5663005a4d404783dc74f4252769b0d5f42856" +checksum = "241eaef5fd12c88705a01fc1066c48c4b36e0dd4377dcdc7ec3942cea7a69956" [[package]] name = "lock_api" @@ -2220,6 +2189,15 @@ dependencies = [ "time", ] +[[package]] +name = "potential_utf" +version = "0.1.2" +source = "registry+https://github.com/rust-lang/crates.io-index" +checksum = "e5a7c30837279ca13e7c867e9e40053bc68740f988cb07f7ca6df43cc734b585" +dependencies = [ + "zerovec", +] + [[package]] name = "powerfmt" version = "0.2.0" @@ -2232,7 +2210,7 @@ version = "0.2.21" source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "85eae3c4ed2f50dcfe72643da4befc30deadb458a9b590d720cde2f2b1e97da9" dependencies = [ - "zerocopy 0.8.25", + "zerocopy", ] [[package]] @@ -2397,7 +2375,7 @@ version = "0.9.3" source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "99d9a13982dcf210057a8a78572b2217b667c3beacbf3a0d8b454f6f82837d38" dependencies = [ - "getrandom 0.3.2", + "getrandom 0.3.3", ] [[package]] @@ -2611,9 +2589,9 @@ dependencies = [ [[package]] name = "rustls-webpki" -version = "0.103.2" +version = "0.103.3" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "7149975849f1abb3832b246010ef62ccc80d3a76169517ada7188252b9cfb437" +checksum = "e4a72fe2bcf7a6ac6fd7d0b9e5cb68aeb7d4c0a0271730218b3e92d43b4eb435" dependencies = [ "ring", "rustls-pki-types", @@ -3231,7 +3209,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "7437ac7763b9b123ccf33c338a5cc1bac6f69b45a136c19bdd8a65e3916435bf" dependencies = [ "fastrand", - "getrandom 0.3.2", + "getrandom 0.3.3", "once_cell", "rustix", "windows-sys 0.59.0", @@ -3347,9 +3325,9 @@ dependencies = [ [[package]] name = "tinystr" -version = "0.7.6" +version = "0.8.1" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "9117f5d4db391c1cf6927e7bea3db74b9a1c1add8f7eda9ffd5364f40f57b82f" +checksum = "5d4f6d1145dcb577acf783d4e601bc1d76a13337bb54e6233add580b07344c8b" dependencies = [ "displaydoc", "zerovec", @@ -3652,12 +3630,6 @@ version = "0.7.6" source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "09cc8ee72d2a9becf2f2febe0205bbed8fc6615b7cb429ad062dc7b7ddd036a9" -[[package]] -name = "utf16_iter" -version = "1.0.5" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "c8232dd3cdaed5356e0f716d285e4b40b932ac434100fe9b7e0e8e935b9e6246" - [[package]] name = "utf8_iter" version = "1.0.4" @@ -4106,17 +4078,11 @@ dependencies = [ "bitflags 2.9.0", ] -[[package]] -name = "write16" -version = "1.0.0" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "d1890f4022759daae28ed4fe62859b1236caebfc61ede2f63ed4e695f3f6d936" - [[package]] name = "writeable" -version = "0.5.5" +version = "0.6.1" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "1e9df38ee2d2c3c5948ea468a8406ff0db0b29ae1ffde1bcf20ef305bcc95c51" +checksum = "ea2f10b9bb0928dfb1b42b65e1f9e36f7f54dbdf08457afefb38afcdec4fa2bb" [[package]] name = "xdg" @@ -4141,9 +4107,9 @@ checksum = "cfe53a6657fd280eaa890a3bc59152892ffa3e30101319d168b781ed6529b049" [[package]] name = "yoke" -version = "0.7.5" +version = "0.8.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "120e6aef9aa629e3d4f52dc8cc43a015c7724194c97dfaf45180d2daf2b77f40" +checksum = "5f41bb01b8226ef4bfd589436a297c53d118f65921786300e427be8d487695cc" dependencies = [ "serde", "stable_deref_trait", @@ -4153,9 +4119,9 @@ dependencies = [ [[package]] name = "yoke-derive" -version = "0.7.5" +version = "0.8.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "2380878cad4ac9aac1e2435f3eb4020e8374b5f13c296cb75b4620ff8e229154" +checksum = "38da3c9736e16c5d3c8c597a9aaa5d1fa565d0532ae05e27c24aa62fb32c0ab6" dependencies = [ "proc-macro2", "quote", @@ -4163,33 +4129,13 @@ dependencies = [ "synstructure", ] -[[package]] -name = "zerocopy" -version = "0.7.35" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "1b9b4fd18abc82b8136838da5d50bae7bdea537c574d8dc1a34ed098d6c166f0" -dependencies = [ - "zerocopy-derive 0.7.35", -] - [[package]] name = "zerocopy" version = "0.8.25" source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "a1702d9583232ddb9174e01bb7c15a2ab8fb1bc6f227aa1233858c351a3ba0cb" dependencies = [ - "zerocopy-derive 0.8.25", -] - -[[package]] -name = "zerocopy-derive" -version = "0.7.35" -source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "fa4f8080344d4671fb4e831a13ad1e68092748387dfc4f55e356242fae12ce3e" -dependencies = [ - "proc-macro2", - "quote", - "syn", + "zerocopy-derive", ] [[package]] @@ -4231,10 +4177,21 @@ source = "registry+https://github.com/rust-lang/crates.io-index" checksum = "ced3678a2879b30306d323f4542626697a464a97c0a07c9aebf7ebca65cd4dde" [[package]] -name = "zerovec" -version = "0.10.4" +name = "zerotrie" +version = "0.2.2" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "aa2b893d79df23bfb12d5461018d408ea19dfafe76c2c7ef6d4eba614f8ff079" +checksum = "36f0bbd478583f79edad978b407914f61b2972f5af6fa089686016be8f9af595" +dependencies = [ + "displaydoc", + "yoke", + "zerofrom", +] + +[[package]] +name = "zerovec" +version = "0.11.2" +source = "registry+https://github.com/rust-lang/crates.io-index" +checksum = "4a05eb080e015ba39cc9e23bbe5e7fb04d5fb040350f99f34e338d5fdd294428" dependencies = [ "yoke", "zerofrom", @@ -4243,9 +4200,9 @@ dependencies = [ [[package]] name = "zerovec-derive" -version = "0.10.3" +version = "0.11.1" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "6eafa6dfb17584ea3e2bd6e76e0cc15ad7af12b09abdd1ca55961bed9b1063c6" +checksum = "5b96237efa0c878c64bd89c436f661be4e46b2f3eff1ebb976f7ef2321d2f58f" dependencies = [ "proc-macro2", "quote", diff --git a/backend/src/ron_extractor.rs b/backend/src/ron_extractor.rs index d5a586e..24fe907 100644 --- a/backend/src/ron_extractor.rs +++ b/backend/src/ron_extractor.rs @@ -22,12 +22,12 @@ where async fn from_request(req: Request, state: &S) -> Result { match req.headers().get(header::CONTENT_TYPE) { Some(content_type) => { - if content_type != ron_utils::RON_CONTENT_TYPE { + if content_type != common::consts::MIME_TYPE_RON { return Err(ron_utils::ron_error( StatusCode::BAD_REQUEST, &format!( "Invalid content type, must be {:?}", - ron_utils::RON_CONTENT_TYPE + common::consts::MIME_TYPE_RON ), ) .into_response()); diff --git a/backend/src/ron_utils.rs b/backend/src/ron_utils.rs index 19caf89..f9cca7b 100644 --- a/backend/src/ron_utils.rs +++ b/backend/src/ron_utils.rs @@ -1,6 +1,6 @@ use axum::{ body::Bytes, - http::{HeaderValue, StatusCode, header}, + http::{StatusCode, header}, response::{ErrorResponse, IntoResponse, Response}, }; use common::ron_api; @@ -9,8 +9,6 @@ use serde::{Serialize, de::DeserializeOwned}; use crate::consts; -pub const RON_CONTENT_TYPE: HeaderValue = HeaderValue::from_static("application/ron"); - #[derive(Debug, Serialize, Clone)] pub struct RonError { pub error: String, @@ -21,7 +19,7 @@ impl axum::response::IntoResponse for RonError { match ron_api::to_string(&self) { Ok(ron_as_str) => ( StatusCode::BAD_REQUEST, - [(header::CONTENT_TYPE, RON_CONTENT_TYPE)], + [(header::CONTENT_TYPE, common::consts::MIME_TYPE_RON)], ron_as_str, ) .into_response(), @@ -39,7 +37,7 @@ impl From for Response { pub fn ron_error(status: StatusCode, message: &str) -> impl IntoResponse { ( status, - [(header::CONTENT_TYPE, RON_CONTENT_TYPE)], + [(header::CONTENT_TYPE, common::consts::MIME_TYPE_RON)], RonError { error: message.to_string(), }, @@ -67,7 +65,7 @@ where match ron_api::to_string(&ron) { Ok(ron_as_str) => ( status, - [(header::CONTENT_TYPE, RON_CONTENT_TYPE)], + [(header::CONTENT_TYPE, common::consts::MIME_TYPE_RON)], ron_as_str, ) .into_response(), diff --git a/backend/src/services/mod.rs b/backend/src/services/mod.rs index 8861ef4..9f324aa 100644 --- a/backend/src/services/mod.rs +++ b/backend/src/services/mod.rs @@ -31,7 +31,7 @@ pub async fn ron_error_to_html( let response = next.run(req).await; if let Some(content_type) = response.headers().get(header::CONTENT_TYPE) { - if content_type == ron_utils::RON_CONTENT_TYPE { + if content_type == common::consts::MIME_TYPE_RON { let message = match body::to_bytes(response.into_body(), usize::MAX).await { Ok(bytes) => String::from_utf8(bytes.to_vec()).unwrap_or_default(), Err(error) => error.to_string(), diff --git a/common/Cargo.toml b/common/Cargo.toml index 5299ea7..8570dee 100644 --- a/common/Cargo.toml +++ b/common/Cargo.toml @@ -6,6 +6,7 @@ edition = "2024" [dependencies] ron = "0.10" +http = "1.3" serde = { version = "1.0", features = ["derive"] } chrono = { version = "0.4", features = ["serde"] } strum = { version = "0.27", features = ["derive"] } diff --git a/common/src/consts.rs b/common/src/consts.rs index b8b09e7..b91c3db 100644 --- a/common/src/consts.rs +++ b/common/src/consts.rs @@ -1,5 +1,11 @@ +use http::HeaderValue; + pub const MIN_PASSWORD_SIZE: usize = 8; pub const COOKIE_DARK_THEME: &str = "dark_theme"; pub const GET_PARAMETER_USER_MESSAGE: &str = "user_message"; pub const GET_PARAMETER_USER_MESSAGE_LEVEL: &str = "user_message_icon"; + +// It should be "application/ron" but firefox can't display the response propertly (base64) +// in the dev tool. +pub const MIME_TYPE_RON: HeaderValue = HeaderValue::from_static("text/ron"); diff --git a/frontend/src/request.rs b/frontend/src/request.rs index 4436a25..40c1005 100644 --- a/frontend/src/request.rs +++ b/frontend/src/request.rs @@ -30,7 +30,6 @@ pub enum Error { type Result = std::result::Result; const CONTENT_TYPE: &str = "Content-Type"; -const CONTENT_TYPE_RON: &str = "application/ron"; async fn req_with_body( api_name: &str, @@ -42,7 +41,10 @@ where U: Serialize, { let url = format!("/ron-api/{}", api_name); - let request_builder = method_fn(&url).header(CONTENT_TYPE, CONTENT_TYPE_RON); + let request_builder = method_fn(&url).header( + CONTENT_TYPE, + common::consts::MIME_TYPE_RON.to_str().unwrap(), + ); send_req(request_builder.body(ron_api::to_string(body)?)?).await } @@ -57,7 +59,10 @@ where { let mut url = format!("/ron-api/{}?", api_name); serde_html_form::ser::push_to_string(&mut url, params).unwrap(); - let request_builder = method_fn(&url).header(CONTENT_TYPE, CONTENT_TYPE_RON); + let request_builder = method_fn(&url).header( + CONTENT_TYPE, + common::consts::MIME_TYPE_RON.to_str().unwrap(), + ); send_req(request_builder.build()?).await }