Merge pull request #21 from dvdsk/fix-dep-soundness

adds changelog, mitigate soundness in a dep

CHANGELOG.md

@@ -0,0 +1,11 @@
+# Changelog
+
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [0.1.1] - 2023-04-20
+
+### Fixed
+ - Resolved soundness issue by replacing dependency `atty` with `is-terminal`

README.md

@@ -1,5 +1,10 @@
 > **Certificate renewal, with advanced diagnostics without installing anything**
 
+See also:
+ - [Install](#Install)
+ - [Example](#Example)
+ - [Changelog](CHANGELOG.md)
+
 Certificate renewal can be quite fragile. This as a renewal tool that runs without install and does not need anything installed. If anything goes south during renewal it does not just report an error. It will try and find out what is wrong and give you a detailed report. For certificate renewal we use [instant-acme](https://crates.io/crates/instant-acme). 
 
 ## Features 

main/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "renewc"
-version = "0.1.0"
+version = "0.1.1"
 edition = "2021"
 description = "Certificate renewal, with advanced diagnostics without installing anything"
 license = "Apache-2.0 OR MIT"
@@ -26,7 +26,7 @@ hyper = { version = "0.14", features = ["client"] }
 itertools = "0.10"
 
 haproxy-config = "0.4"
-atty = "0.2"
+is-terminal = "0.4"
 shared_memory = "0.12.4"
 
 [dev-dependencies]

main/src/lib.rs

@@ -69,7 +69,8 @@ pub async fn run(config: impl Into<Config>, debug: bool) -> eyre::Result<()> {
 
 #[must_use]
 fn exit_requested(question: &str) -> bool {
-    if atty::isnt(atty::Stream::Stdin) {
+    use is_terminal::IsTerminal;
+    if !std::io::stdin().is_terminal() {
         true; // not a terminal, take the safe option
     }