Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@georglauterbach
Copy link
Member

Description

Fixes #4180

Type of change

  • This change is a documentation update

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code

No CHANGELOG.md entry required.

@georglauterbach georglauterbach added area/scripts kind/improvement Improve an existing feature, configuration file or the documentation area/documentation labels Oct 29, 2024
@georglauterbach georglauterbach added this to the v15.0.0 milestone Oct 29, 2024
@georglauterbach georglauterbach self-assigned this Oct 29, 2024
@georglauterbach georglauterbach enabled auto-merge (squash) October 29, 2024 23:03
Comment on lines +58 to +60
${LWHITE}setup config dkim keytype ed25519 domain edward.com selector elliptic-test${RESET}
Generate the DKIM key using the ED25519 elliptic curve for the domain
edward.com and the selector elliptic-test.
Copy link
Member

Choose a reason for hiding this comment

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

  • Any reason to use edward.com instead of example.com placeholder?
  • A selector example was already provided and probably isn't adding much here?

Revised suggestion if you agree:

Suggested change
${LWHITE}setup config dkim keytype ed25519 domain edward.com selector elliptic-test${RESET}
Generate the DKIM key using the ED25519 elliptic curve for the domain
edward.com and the selector elliptic-test.
${LWHITE}setup config dkim keytype ed25519 domain example.com${RESET}
Generate a DKIM key for the domain 'example.com' but with ED25519 instead of RSA.

Copy link
Member Author

Choose a reason for hiding this comment

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

Please don't hesitate to adjust the code; the PR has been merged with auto-merge. I used "edward.com" because of ed25519 is based on Edward's Elliptical curve; but we can stick with example.com and remove the selector.

@georglauterbach georglauterbach merged commit 662afec into master Oct 30, 2024
@georglauterbach georglauterbach deleted the rspamd-dkim-keytype branch October 30, 2024 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation area/scripts kind/improvement Improve an existing feature, configuration file or the documentation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

bug report: built-in documentation does not give an example for rspamd-dkim's keytype

3 participants