fix: Changed way of downloading and using assets in pre 1.6 versions

onelauncher/src/game/client.rs

@@ -69,6 +69,7 @@ pub trait ClientTrait<'a>: Send + Sync {
         let manifest = &self.get_manifest().minecraft_manifest;
 
         let args = get_arguments(manifest)?
+            .replace("${game_assets}", "${assets_root}")
             .replace("${auth_player_name}", &info.account.username)
             .replace("${version_name}", &info.setup.version)
             .replace("${game_directory}", info.setup.game_dir.to_str().unwrap())

onelauncher/src/game/clients/vanilla.rs

@@ -30,26 +30,31 @@ impl<'a> ClientTrait<'a> for VanillaClient<'a> {
 	async fn setup(&self) -> crate::Result<SetupInfo> {
         let manifest = &self.manifest.minecraft_manifest;
 
+        // Create game directory
+        let game_dir = self.cluster.dir()?.join("game");
+        fs::create_dir_all(&game_dir)?;
+
         // Install everything
         let client_path = install_game(manifest).await?;
         let mut libraries = install_libraries(manifest).await?;
-        let assets = install_assets(manifest).await?;
+
+        // Resources dir used for legacy versions 
+        // with map_to_resources set to true in their asset indexes
+        let resources_dir = game_dir.join("resources");
+        let (asset_index, assets_dir) = install_assets(manifest, &resources_dir).await?;
 
         // Append client path to the libraries string at the end
         libraries.push_str(constants::LIBRARY_SPLITTER);
         libraries.push_str(client_path.to_str().unwrap());
         
-        // Create game directory
-        let game_dir = self.cluster.dir()?.join("game");
-        fs::create_dir_all(&game_dir)?;
 
 		Ok(SetupInfo {
             version: manifest.version.clone(),
             libraries,
             natives_dir: dirs::natives_dir()?,
             game_dir,
-            assets_dir: dirs::assets_dir()?,
-            asset_index: assets,
+            assets_dir,
+            asset_index,
         })
 	}
 }
@@ -130,7 +135,7 @@ pub async fn install_natives(natives: Vec<&Library>) -> crate::Result<()> {
     Ok(())
 }
 
-pub async fn install_assets(manifest: &MinecraftManifest) -> crate::Result<String> {
+pub async fn install_assets(manifest: &MinecraftManifest, resources_dir: &PathBuf) -> crate::Result<(String, PathBuf)> {
     let assets = dirs::assets_dir()?;
     let objects = assets.join("objects");
     let indexes = assets.join("indexes");
@@ -145,10 +150,16 @@ pub async fn install_assets(manifest: &MinecraftManifest) -> crate::Result<Strin
     }
 
     let contents = serde_json::from_str::<AssetIndexFile>(&fs::read_to_string(&index)?)?;
-    for (_, asset) in contents.objects {
+    for (path, asset) in contents.objects {
         let hash = asset.hash.clone();
         let short = &hash[..2];
-        let file = objects.join(&short).join(&hash);
+        
+        let file = if contents.map_to_resources {
+            resources_dir.join(path)
+        } else {
+            objects.join(&short).join(&hash)
+        };
+        
         fs::create_dir_all(file.parent().ok_or(anyhow!("Couldn't get asset parent"))?)?;
 
         if !file.exists() {
@@ -157,7 +168,10 @@ pub async fn install_assets(manifest: &MinecraftManifest) -> crate::Result<Strin
         }
     }
 
-    Ok(manifest.asset_index.id.clone())
+    Ok((
+        manifest.asset_index.id.clone(),
+        if contents.map_to_resources { resources_dir.clone() } else { assets }
+    ))
 }
 
 #[derive(Clone, Serialize, Deserialize)]
@@ -217,7 +231,11 @@ pub async fn retrieve_version_manifest(url: &str) -> crate::Result<MinecraftMani
 		.get(url)
 		.send()
 		.await?
-		.json::<MinecraftManifest>()
-		.await?;
-	Ok(manifest)
+		.json::<MinecraftManifest>().await;
+
+    if let Err(err) = manifest {
+        println!("{:#?}", err);
+        return Err(err.into());
+    }
+	Ok(manifest.unwrap())
 }

onelauncher/src/game/minecraft.rs

@@ -23,12 +23,14 @@ pub struct MinecraftManifest {
     pub downloads: Downloads,
     pub java_version: JavaVersion,
     pub libraries: Vec<Library>,
-    pub logging: Logging,
     pub main_class: String,
     pub release_time: String,
-
+    
     #[serde(rename = "id")]
     pub version: String,
+
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub logging: Option<Logging>,
     
     #[serde(rename = "type")]
     pub release_type: ReleaseType,
@@ -91,7 +93,7 @@ pub struct ModernArgumentRuleItem {
 }
 
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
-#[serde(rename_all = "kebab-case")]
+#[serde(rename_all = "snake_case")]
 pub enum ReleaseType {
     Snapshot,
     Release,
@@ -268,9 +270,13 @@ pub struct File {
 
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub struct AssetIndexFile {
+    #[serde(default = "default_false")]
+    pub map_to_resources: bool,
     pub objects: HashMap<String, AssetObject>,
 }
 
+fn default_false() -> bool { false }
+
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub struct AssetObject {
     pub hash: String,

src/ui/pages/Home.tsx

@@ -196,8 +196,8 @@ function HomePage() {
 
 	async function newInstance() {
 		await createCluster({
-			name: 'Modern cluster',
-			version: '1.8.9',
+			name: '1.0 cluster',
+			version: 'a1.0.11',
 			client: {
 				type: 'Vanilla',
 			},