Skip to content

himalaya: fix smtp-starttls option#2744

Merged
sumnerevans merged 1 commit intonix-community:masterfrom
NomisIV:patch-1
Feb 21, 2022
Merged

himalaya: fix smtp-starttls option#2744
sumnerevans merged 1 commit intonix-community:masterfrom
NomisIV:patch-1

Conversation

@NomisIV
Copy link
Copy Markdown
Contributor

@NomisIV NomisIV commented Feb 21, 2022

Description

The smtp-starttls option was previously to the value of account.imap.tls.useStartTls, when I believe account.smtp.tls.useStartTls should be used instead

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.

    • Added myself and the module files to .github/CODEOWNERS.

Copy link
Copy Markdown
Contributor

@sumnerevans sumnerevans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks right to me

The `smtp-starttls` option was previously to the value of `account.imap.tls.useStartTls`, when `account.smtp.tls.useStartTls` should be used instead
@sumnerevans sumnerevans merged commit 69536af into nix-community:master Feb 21, 2022
@NomisIV NomisIV deleted the patch-1 branch February 21, 2022 22:09
Kranzes pushed a commit to Kranzes/home-manager that referenced this pull request Feb 22, 2022
The `smtp-starttls` option was previously to the value of `account.imap.tls.useStartTls`, when `account.smtp.tls.useStartTls` should be used instead
@teto teto mentioned this pull request Aug 22, 2022
7 tasks
teto pushed a commit to teto/home-manager that referenced this pull request Aug 22, 2022
The `smtp-starttls` option was previously to the value of `account.imap.tls.useStartTls`, when `account.smtp.tls.useStartTls` should be used instead
spacekookie pushed a commit to spacekookie/home-manager that referenced this pull request Feb 10, 2023
The `smtp-starttls` option was previously to the value of `account.imap.tls.useStartTls`, when `account.smtp.tls.useStartTls` should be used instead
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants