Update Rust to 1.87.0 (#772)

Author: oguzkocer

Makefile

@@ -4,7 +4,7 @@
 docker_container_repo_dir=/app
 
 # Common docker options
-rust_docker_container := public.ecr.aws/docker/library/rust:1.86.0
+rust_docker_container := public.ecr.aws/docker/library/rust:1.87.0
 
 docker_opts_shared := --rm -v "$(PWD)":$(docker_container_repo_dir) -w $(docker_container_repo_dir)
 rust_docker_run := docker run -v $(PWD):/$(docker_container_repo_dir) -w $(docker_container_repo_dir) -it -e TEST_ALL_PLUGINS -e CARGO_HOME=/app/.cargo $(rust_docker_container)

wp_contextual/src/wp_contextual.rs

@@ -157,7 +157,9 @@ fn parse_fields(
                                 .into_syn_error(attr.meta.span()))
                         }
                     } else {
-                        Ok(WpParsedAttr::ExternalAttr { attr: attr.clone() })
+                        Ok(WpParsedAttr::ExternalAttr {
+                            attr: Box::new(attr.clone()),
+                        })
                     }
                 })
                 .collect::<Result<Vec<WpParsedAttr>, syn::Error>>()?;
@@ -394,7 +396,7 @@ impl GeneratedContextualField {
                         .filter_map(|parsed_attr| {
                             // The generated field should only contain external attributes
                             if let WpParsedAttr::ExternalAttr { attr } = parsed_attr {
-                                Some(attr.to_owned())
+                                Some(*attr.clone())
                             } else {
                                 None
                             }
@@ -715,7 +717,7 @@ enum WpParsedAttr {
     ParsedWpContextualField,
     ParsedWpContextualOption,
     ParsedWpContext { contexts: Vec<WpContextAttr> },
-    ExternalAttr { attr: syn::Attribute },
+    ExternalAttr { attr: Box<syn::Attribute> },
 }
 
 #[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord)]

wp_derive_request_builder/src/lib.rs

@@ -27,12 +27,11 @@ fn read_crate_config() -> CrateConfig {
     let manifest_dir = env::var("CARGO_MANIFEST_DIR")
         .expect("Crate config can't be found without the `CARGO_MANIFEST_DIR` environment varible");
     let file_path = format!("{}/wp_derived_request.toml", manifest_dir);
-    let contents = match fs::read_to_string(&file_path) {
+    match fs::read_to_string(&file_path) {
         Ok(c) => toml::from_str(c.as_str())
             .unwrap_or_else(|e| panic!("'{}' is not formatted correctly:\n{:#?}", file_path, e)),
         Err(_) => {
             panic!("{} is missing", file_path);
         }
-    };
-    contents
+    }
 }

wp_derive_request_builder/src/variant_attr.rs

@@ -89,10 +89,10 @@ impl ParsedVariantAttribute {
         if let syn::Meta::List(meta_list) = attr.meta {
             Ok(meta_list)
         } else {
-            Err(
-                ItemVariantAttributeParseError::MetaInWrongFormat { meta: attr.meta }
-                    .into_syn_error(input.span()),
-            )
+            Err(ItemVariantAttributeParseError::MetaInWrongFormat {
+                meta: Box::new(attr.meta),
+            }
+            .into_syn_error(input.span()))
         }
     }
 
@@ -373,7 +373,7 @@ enum ItemVariantAttributeParseError {
     #[error("Only a single attribute is supported")]
     MoreThanOneOuterAttr,
     #[error("Expecting a syn::Meta::List found {:?}", meta)]
-    MetaInWrongFormat { meta: syn::Meta },
+    MetaInWrongFormat { meta: Box<syn::Meta> },
     #[error(
         "Expecting key value pairs (url = \"\", params = FooParam, output = FooOutput, filter_by = FooSparseField)"
     )]

wp_rs_web/Dockerfile

@@ -1,4 +1,4 @@
-FROM rust:1.86.0 AS builder
+FROM rust:1.87.0 AS builder
 
 WORKDIR /app