Merge branch 'dont-limit-coredump' into 'main'

remove manual limitation of coredump size

Closes #598

See merge request tpo/core/arti!757

Cargo.lock

@@ -230,7 +230,6 @@ dependencies = [
  "futures",
  "pin-project",
  "rand 0.8.5",
- "rlimit",
  "serde",
  "tokio",
  "tor-checkable",

crates/arti-testing/Cargo.toml

@@ -27,7 +27,6 @@ config = { version = "0.13", default-features = false }
 futures = "0.3.14"
 pin-project = "1"
 rand = "0.8"
-rlimit = "0.8.3"
 serde = { version = "1.0.103", features = ["derive"] }
 tokio = { version = "1.7", features = ["signal", "macros"] }
 tor-checkable = { path = "../tor-checkable", version = "0.3.0", features = ["experimental-api"] }

crates/arti/src/process.rs

@@ -43,10 +43,6 @@ pub(crate) fn enable_process_hardening() -> anyhow::Result<()> {
         return Ok(());
     }
 
-    #[cfg(unix)]
-    rlimit::setrlimit(rlimit::Resource::CORE, 0, 0)
-        .context("Problem while disabling core dumps")?;
-
     secmem_proc::harden_process_std_err().context("Problem while hardening process")?;
 
     Ok(())