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

Fixes #179: Add missing /tool/e-mail parameter "tls" #180

Merged

Conversation

derdeagle
Copy link
Contributor

SUMMARY

Fixes #179: Add missing /tool/e-mail parameter "tls"

In the
documentation the parameter "tls" is mentioned but it cannot be used. The old "start-tls" parameter is not mentioned there (only on the old documentation). Trying to use the paramter "tls" lead to the erro message FAILED! => {"changed": false, "msg": "Unknown key \"tls\"."}.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

community.routeros.api_modify

ADDITIONAL INFORMATION

None.

In the
[documentation](https://help.mikrotik.com/docs/display/ROS/E-mail#Email-Properties) the parameter "tls" is mentioned but it cannot be used. The old "start-tls" parameter is not mentioned there (only on the [old documentation](https://wiki.mikrotik.com/wiki/Manual:Tools/email#Properties)).
Trying to use the paramter "tls" lead to the erro message `FAILED! => {"changed": false, "msg": "Unknown key \"tls\"."}`.
@codecov
Copy link

codecov bot commented Jun 12, 2023

Codecov Report

Merging #180 (b03f662) into main (9cfb31a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #180   +/-   ##
=======================================
  Coverage   84.57%   84.57%           
=======================================
  Files          32       32           
  Lines        3850     3850           
  Branches      710      710           
=======================================
  Hits         3256     3256           
  Misses        456      456           
  Partials      138      138           
Flag Coverage Δ
integration 66.86% <ø> (ø)
sanity 22.23% <ø> (ø)
units 84.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
plugins/module_utils/_api_data.py 96.92% <ø> (ø)

Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Could you please add a changelog fragment? Thanks.

@derdeagle
Copy link
Contributor Author

Thanks for reviewing and sorry I missed that. I have added the changelog fragment.

@felixfontein
Copy link
Collaborator

sorry I missed that

No worries, that's what reviews are for :)

As suggested by felixfontein

Co-authored-by: Felix Fontein <[email protected]>
@felixfontein felixfontein merged commit af28e64 into ansible-collections:main Jun 19, 2023
@felixfontein
Copy link
Collaborator

@derdeagle thanks for your contribution!

@derdeagle
Copy link
Contributor Author

@felixfontein Thank you, too.

@derdeagle derdeagle deleted the fix-email-tls-parameter branch June 19, 2023 11:13
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.

/tool/e-mail "tls" parameter cannot be set
2 participants