Outlook bug

[Diego-H-S]

Summary

• In the subject parameter occasionally appears an extra separator, which has been solved by replacing it with a space.
• When the length of the receivers parameter is longer than 8000 (maximum length for creating a varchar) the string is truncated to the latest email added to this receivers parameter.

Importance

It happened in the past but I would be necessary to avoid similar situations in the future.

Checklist

This PR:

• follows the guidelines laid out in CONTRIBUTING.md
• links relevant issue(s)
• adds/updates tests (if appropriate)
• adds/updates docstrings (if appropriate)
• adds an entry in CHANGELOG.md

[Rafalz13]

• Added comments.
• Please add tests for this functionality with 8000.

[Rafalz13]

My suggestion - what do you think about using only 1 if to this case?

Suggested change

	if sum(list(map(len, [recivers, add_string]))) >= 8000:
	break
	else:
	recivers += add_string
	if sum(list(map(len, [recivers, add_string]))) < 8000:
	recivers += add_string

[Diego-H-S]

The end is the same, because, I need to break the for loop when it reaches 8000, after that limit you are iterating without any purpose.
If I use your sentence, then I would need an else: break which is basically the same as it is already there.

[Rafalz13]

Please add or modify tests for new features:

• break if >= 8000
• handling receivers

[Diego-H-S]

created a new parameter and updated test file.

[Rafalz13]

Please reduce logs to only directories where we can find some messages. In the previous version, a user was able to see everything. (It was a long log message)

[Diego-H-S]

Please reduce logs to only directories where we can find some messages. In the previous version, a user was able to see everything. (It was a long log message)

I have updated the source file to print out only folders with some messages.