Session logging bugfix and small improvement

[dirkjanm]

This fixes #8597 by only stripping newline characters from the string instead of stripping the last character no matter what (which cuts short user commands).

It also adds the session name (usually the number) to the file, to prevent multiple sessions to the same IP writing to the same file and making things confusing.

Verification

• Start msfconsole
• Enable session logging setg SessionLogging true
• Spawn a meterpreter
• The session log file is created, and the commands in there do not get cut off
• The filename contains the session ID
• Spawn a second meterpreter on the same host
• A second session file gets generated, instead of output from both sessions being written to the same file

[busterb]

Looks great, thanks @dirkjanm

[busterb]

Release Notes

This fix resolves a bug in session logging that caused the final character of a line to be removed from the log. It also adds the session name (usually the number) to the file, which prevents multiple sessions to the same IP from writing to the same file.