amidaware / tacticalrmm

A remote monitoring & management tool, built with Django, Vue and Go.
https://docs.tacticalrmm.com
Other
3.24k stars 443 forks source link

save full output of check history #1933

Open P6g9YHK6 opened 3 months ago

P6g9YHK6 commented 3 months ago

Server Info (please complete the following information):

Installation Method:

Agent Info (please complete the following information):

Describe the bug The STDOUT and STDERR is not visible if more than a couple of charr long in the history nor a full output can be found in the CSV export

To Reproduce Steps to reproduce the behavior:

  1. Go to check history
  2. Click on any point in history that has a long output
  3. See error

Expected behavior either allow to copy the output on click or make it show the whole thing on click and additionally add the STDOUT STDERR to the CSV export

Screenshots image

Additional context https://discord.com/channels/736478043522072608/889513877124042762/1265966553816367168

wh1te909 commented 3 months ago

removed bug label and added enhancement as technically this is by design, we are intentionally truncating the output before it's saved in the database.

P6g9YHK6 commented 1 week ago

if the data start's being too bulky it should maybe be saved into another DB because i am not sure postgress would survive all the checks in full :D

silversword411 commented 1 week ago

How many characters is it currently saving? Maybe an increase to a reasonable 256-512 would be a good quick fix.

P6g9YHK6 commented 1 week ago

How many characters is it currently saving? Maybe an increase to a reasonable 256-512 would be a good quick fix.

this and on top fixing the csv export feature would indeed be a good patchwork https://github.com/amidaware/tacticalrmm/issues/2025