fix: it's no error if credential helpers don't receive context as input.

They might ignore all input and output hardcoded credentials for good reason,
so we don't validate writes to the helper similarly to how git does it.

Note that we also don't care if credentials are stored or erased for similar reasons.
I would happily make the case that we should be more adamant about error handling in that
case, but typically we check error codes and that's the only indicator for success or failure.

Related to rust-lang/cargo#11821

Author: Byron

gix-credentials/src/helper/invoke.rs

@@ -8,8 +8,9 @@ impl Action {
         match self {
             Action::Get(ctx) => ctx.write_to(write),
             Action::Store(last) | Action::Erase(last) => {
-                write.write_all(last)?;
-                write.write_all(&[b'\n'])
+                write.write_all(last).ok();
+                write.write_all(&[b'\n']).ok();
+                Ok(())
             }
         }
     }

gix-credentials/src/protocol/context/serde.rs

@@ -21,7 +21,7 @@ mod write {
                 if let Some(value) = value {
                     validate(key, value.as_slice().into())
                         .map_err(|err| std::io::Error::new(std::io::ErrorKind::Other, err))?;
-                    write_key(&mut out, key, value.as_ref())?;
+                    write_key(&mut out, key, value.as_ref()).ok();
                 }
             }
             for (key, value) in [
@@ -33,7 +33,7 @@ mod write {
                 if let Some(value) = value {
                     validate(key, value.as_str().into())
                         .map_err(|err| std::io::Error::new(std::io::ErrorKind::Other, err))?;
-                    write_key(&mut out, key, value.as_bytes().as_bstr())?;
+                    write_key(&mut out, key, value.as_bytes().as_bstr()).ok();
                 }
             }
             Ok(())