6530956920
86%

Ran 16 Oct 2023 08:32AM UTC

Jobs 0

Files 0

Run time –

Badge

Embed ▾

pending completion

Build # 6530956920

Build Type

push

github

Committed by

web-flow

Commit Message

Merge pull request #4150 from esl/fix-starttls-disabled

When the TLS section is missing, then according to the documentation STARTTLS should be rejected. This was not the case: the feature was advertised, and a TLS upgrade performed by a client (who was informed about the support) resulted in a crash.

This PR fixes these issues:

    STARTTLS is only advertised when enabled.
    Upgrade attempt results in a correct failure element, as described in RFC 6120

The tests are updated to check these conditions, and to correctly verify the features for optional and required STARTTLS as well.

The bugs were discovered accidentally when checking esl/mongooseim-docker#42. The statement about TLS disabled by default is changed in the docs, because it most likely confused the reporter of that issue.

Run Details

Source Files on build 6530956920

Detailed source file information is not available for this build.

esl / MongooseIM / 6530956920
86%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 6530956920

esl / MongooseIM / 6530956920 86%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 6530956920

esl / MongooseIM / 6530956920
86%

README BADGES
x