Skip to content
Snippets Groups Projects

Resolve "cli: keys modify: ask salt and password on password prompt for security"

1 unresolved thread

Closes #201 (closed)

Edited by dvermd

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Éloïs
  • Éloïs
  • Éloïs
  • Éloïs
  • 261 stdin_mock.expect_get_password().returning(|prompt| {
    262 if prompt.starts_with("Salt:") {
    263 Ok(Secret {
    264 secret: SALT_TEST.to_owned(),
    265 })
    266 } else if prompt.starts_with("Password:") {
    267 Ok(Secret {
    268 secret: PASSWORD_TEST.to_owned(),
    269 })
    270 } else {
    271 Err(std::io::Error::new(
    272 std::io::ErrorKind::InvalidInput,
    273 format!("should not be called with {}", prompt),
    274 ))
    275 }
    276 });
  • Author Owner

    @dvermd, Pense aux #[inline] sur les fonctions qui n'ont qu'une seule instruction :)

  • dvermd added 2 commits

    added 2 commits

    • a35e0def - wip1 sans les test
    • e174c4d5 - [ref] conf: refactor tests and improve readability

    Compare with previous version

  • dvermd added 1 commit

    added 1 commit

    • fc2d94ef - [ref] conf: refactor tests and improve readability

    Compare with previous version

  • Developer

    @librelois la CI ne démarre pas, elle est marquée en skip. Par rapport à la dernière CI sur le commit e174c4d5 j'ai juste squashé 2 commits de refactor

  • Author Owner

    @dvermd Oui c'est tout a fait normal, la CI ne se déclenche que s'il y a des changements de code depuis le dernier push. Il faut toujours que ton dernier commit commence par wip tant que ton historique git n'est pas au propre. Dans le cas présent, je me référerai a la pipeline du push précédent (que j'ai relancé, tu l'avais annulée apparemment).

    Edited by Éloïs
  • dvermd added 4 commits

    added 4 commits

    • fc2d94ef...d22ddd88 - 2 commits from branch dev
    • b8cfcda6 - [feat] core+conf: Replace keys modify salt/password opt by interactive input
    • e73b2ea1 - [ref] conf: refactor tests and improve readability

    Compare with previous version

  • Author Owner

    @dvermd serait tu dispo pour me rejoindre sur mumble maintenant par hazard ? :)

  • Developer

    @librelois c'est une super opti de CI de ne démarrer que s'il y a eu des changements depuis le dernier push. J'ai l'habitude des CI qui démarrent à chaque push sans distinction. J'avais push un historique pas propre, je m'en suis aperçu, j'ai arrêté le CI pour libérer les ressources de la machine, nettoyer l'historique et refaire push.

    Je viens de rebase sur dev

  • closed

  • Please register or sign in to reply
    Loading