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

[CLI+APIclient] Change timeline color for a given timeline #3159

Merged

Conversation

jaegeral
Copy link
Collaborator

Change timeline color

The color is an important setting for a timeline when using the WebUI. To change the color using the CLI timelines color can be used.

This change also adds a setter for timeline.color in the API client that has not been present so far.

Before:

timesketch --sketch 1 --output-format text timelines describe 2
Name: foobar3
Index: 41dde394812d44c1ac1784997d05efed
Status: ready
Event count: 260454
Color: AAAAAA
Name: foobar3
Created: 2024-08-20T14:57:59.047015
Datasources:
	Original filename: win7-x86.plaso
	File on disk: /tmp/4c3c1c5c351b4db285453bff0ecad51e
	Error:

Using it:

timesketch --sketch 1 timelines color 2 BBBBBB

After:

timesketch --sketch 1 --output-format text timelines describe 2
Name: foobar3
Index: 41dde394812d44c1ac1784997d05efed
Status: ready
Event count: 260454
Color: BBBBBB
Name: foobar3
Created: 2024-08-20T14:57:59.047015
Datasources:
	Original filename: win7-x86.plaso
	File on disk: /tmp/4c3c1c5c351b4db285453bff0ecad51e
	Error:

This is the last feature request for #2877

@jaegeral
Copy link
Collaborator Author

Of course it can be discussed if we also accept Hex codes with #AABBCC but I decided against it to keep it simple and avoid mega regex to check values.

Copy link
Collaborator

@jkppr jkppr left a comment

Choose a reason for hiding this comment

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

lgtm

@jaegeral jaegeral merged commit 809ad2d into google:master Aug 22, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants