Skip to content

Commit

Permalink
Revert "fix(core): set correct mimetype for asset protocol streams, c…
Browse files Browse the repository at this point in the history
…loses #5203 (#5210)"

This reverts commit 39443b4.
  • Loading branch information
amrbashir authored and lucasfernog committed Nov 3, 2022
1 parent 23859f7 commit 2d9c2b4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 29 deletions.
5 changes: 0 additions & 5 deletions .changes/asset-protocol-streaming-mime-type.md

This file was deleted.

26 changes: 2 additions & 24 deletions core/tauri/src/manager.rs
Expand Up @@ -507,9 +507,7 @@ impl<R: Runtime> WindowManager<R> {
use crate::api::file::SafePathBuf;
use tokio::io::{AsyncReadExt, AsyncSeekExt};
use url::Position;
let state = self.state();
let asset_scope = state.get::<crate::Scopes>().asset_protocol.clone();
let mime_type_cache = MimeTypeCache::default();
let asset_scope = self.state().get::<crate::Scopes>().asset_protocol.clone();
pending.register_uri_scheme_protocol("asset", move |request| {
let parsed_path = Url::parse(request.uri())?;
let filtered_path = &parsed_path[..Position::AfterPath];
Expand Down Expand Up @@ -624,7 +622,7 @@ impl<R: Runtime> WindowManager<R> {
response = response.header(k, v);
}

let mime_type = mime_type_cache.get_or_insert(&data, &path);
let mime_type = MimeType::parse(&data, &path);
response.mimetype(&mime_type).status(status_code).body(data)
} else {
match crate::async_runtime::safe_block_on(async move { tokio::fs::read(path_).await }) {
Expand Down Expand Up @@ -1429,26 +1427,6 @@ fn request_to_path(request: &tauri_runtime::http::Request, base_url: &str) -> St
}
}

// key is uri/path, value is the store mime type
#[cfg(protocol_asset)]
#[derive(Debug, Clone, Default)]
struct MimeTypeCache(Arc<Mutex<HashMap<String, String>>>);

#[cfg(protocol_asset)]
impl MimeTypeCache {
pub fn get_or_insert(&self, content: &[u8], uri: &str) -> String {
let mut cache = self.0.lock().unwrap();
let uri = uri.to_string();
if let Some(mime_type) = cache.get(&uri) {
mime_type.clone()
} else {
let mime_type = MimeType::parse(content, &uri);
cache.insert(uri, mime_type.clone());
mime_type
}
}
}

#[cfg(test)]
mod tests {
use super::replace_with_callback;
Expand Down

0 comments on commit 2d9c2b4

Please sign in to comment.