Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check for RFC8460, SMTP-TLS reporting (TLS-RPT) #881

Open
mdavids opened this issue Feb 22, 2023 · 5 comments
Open

Check for RFC8460, SMTP-TLS reporting (TLS-RPT) #881

mdavids opened this issue Feb 22, 2023 · 5 comments
Milestone

Comments

@mdavids
Copy link

mdavids commented Feb 22, 2023

Aangezien er gecheckt wordt op STARTTLS / DANE, vroeg ik me af of we niet ook een check moeten toevoegen op het bestaan van SMTP-TLS reporting (https://www.rfc-editor.org/rfc/rfc8460.html)?

We zouden daar een 'informational' van kunnen maken.

Voorbeeld:

dig TXT _smtp._tls.sidn.nl

@WKobes
Copy link
Collaborator

WKobes commented Feb 23, 2023

Related/partial duplicate: #458

@baknu baknu changed the title Check for RFC8460, SMTP-TLS reporting? Check for RFC8460, SMTP-TLS reporting? (TLS-RPT) Jan 29, 2024
@baknu
Copy link
Contributor

baknu commented Jan 29, 2024

Further ideas on this (from a discussion between SR and BK):

  • Subtest for TLS-RPT should be part of email test (https://en.internet.nl/test-mail/).
  • Probably create separate (fourth) category "Reporting" under "Secure mail server connection (STARTTLS and DANE)".
  • Subtest should check for (1) presence and (2) validity of a TLS-RPT DNS record of a tested domain.
  • Working of TLS-RPT subtest should be similar to existing subtests for DMARC and SPF.
  • Failing will result in either an "informational" ℹ️ or a "warning" ⚠️ (to be decided). But not a "bad" ❌ as we currently do not have plans to let this subtest weigh into the score.
  • Preferably a proven Python parser/validator is used for the machinery of the subtest. This was also done for the security.txt subtest that makes use of https://github.com/DigitalTrustCenter/sectxt.
  • Content (test verdict, technical details, test explanation) should be available in English and Dutch.
  • The subtest for TLS-RPT should also be included in the API of the batch version of Internet.nl (http://redocly.github.io/redoc/?url=https://batch.internet.nl/api/batch/openapi.yaml).
  • Practice what you preach: Internet.nl itself should probably also publish a TLS-RPT record. See also: Set up functional mailbox on @internet.nl for reporting used by several standards #957

@baknu baknu changed the title Check for RFC8460, SMTP-TLS reporting? (TLS-RPT) Check for RFC8460, SMTP-TLS reporting (TLS-RPT) Jan 29, 2024
@baknu baknu added this to the backlog milestone Jan 29, 2024
@patrickbenkoetter
Copy link

Further ideas on this (from a discussion between SR and BK):

* Subtest for TLS-RPT should be part of email test (https://en.internet.nl/test-mail/).

* Probably create separate (fourth) category "Reporting" under "Secure mail server connection (STARTTLS and DANE)".

* Subtest should check for (1) presence and (2) validity of a TLS-RPT DNS record of a tested domain.

* Working of TLS-RPT subtest should be similar to existing subtests for DMARC and SPF.

* Failing will result in either an "informational" ℹ️ or a "warning" ⚠️ (to be decided). But not a "bad" ❌ as we currently do not have plans to let this subtest weigh into the score.

* Preferably a proven Python parser/validator is used for the machinery of the subtest. This was also done for the security.txt subtest that makes use of https://github.com/DigitalTrustCenter/sectxt.

* Content (test verdict, technical details, test explanation) should be available in English and Dutch.

* The subtest for TLS-RPT should also be included in the API of the batch version of Internet.nl (http://redocly.github.io/redoc/?url=https://batch.internet.nl/api/batch/openapi.yaml).

* Practice what you preach: Internet.nl itself should probably also publish a TLS-RPT record. See also: [Set up functional mailbox on @internet.nl for reporting used by several standards #957](https://github.com/internetstandards/Internet.nl/issues/957)

@baknu We're all in for your suggestions but we might need some help on the Dutch content once we will have the English done. Mind to help us with that?

@baknu
Copy link
Contributor

baknu commented Jan 30, 2024

@baknu We're all in for your suggestions but we might need some help on the Dutch content once we will have the English done. Mind to help us with that?

Sure, we can help with that.

uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 10, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 10, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 10, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 10, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 10, 2024
…rrect translation markers, add tlsrpt callback
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 10, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 10, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 12, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 25, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 26, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 29, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Feb 29, 2024
@uwekamper
Copy link
Contributor

There is now a PR here: #1300

uwekamper added a commit to sys4/Internet.nl that referenced this issue Apr 8, 2024
uwekamper added a commit to sys4/Internet.nl that referenced this issue Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

5 participants