poshbotio / PoshBot

Powershell-based bot framework
MIT License
536 stars 108 forks source link

Allow config file to set what timezone for logs (And/Or turn off universal time) #193

Open Kickbut101 opened 4 years ago

Kickbut101 commented 4 years ago

Currently in Poshbot.psm1 (Version 12.0.0) line 117 is; [datetime]$DateTime = (Get-Date).ToUniversalTime()

I understand this is probably convenient or useful to some. But I found it troublesome and a little bit annoying to try to figure out why all my timestamps made no sense while troubleshooting.

Expected Behavior

I can easily determine issues with the script by checking timestamps without wondering why they are all way off from my local time

Current Behavior

Time is "off" (at least to me)

Possible Solution

Allow the timestamp to be set to local time or to universaltime.

Your Environment

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

devblackops commented 4 years ago

@Kickbut101 We can probably make that a configuration option with UTC being the default. Make sense?

Kickbut101 commented 4 years ago

@Kickbut101 We can probably make that a configuration option with UTC being the default. Make sense?

That would totally work!