Feature: Better export filenames

[denandz]

Feature Description

When exporting graph content from bloodhound, the filename is always bh-graph.json which leads to a downloads directory full of bh-graph (x).json after any reasonable sized engagement. The export needs a better file naming scheme, even bh-graph-$currentepoch.json would be an improvement. Popping a modal that lets me input a filename, or hit 'enter' and accept a reasonable default, would be even better.

Are you intending to implement this feature?

yes

Current Behavior

Clicking on 'export graph' downloads a file called 'bh-graph.json'

Desired Behavior

Clicking on 'export graph' should either download a file with a unique name by-default, or ideally pop a modal that lets the user input a filename or just hit 'enter' and accept bh-graph-$epoch.json or similar.

Use Case

A thorough review of an environment involves more than just the bloodhound UI, and extracting data from specific cypher queries for further processing or cross-referencing against other data sets is easiest with the export graph functionality.

Having to manually rename files or deal with multiple bh-graph.json files is annoying and unintuitive. A better file naming scheme would be a quality-of-life improvement for Bloodhound users who use BH in a similar way.

Implementation Suggestions

Would involve tweaking the export method to accept a filename parameter and implementing a modal to supply a default filename or user-supplied filename.

BloodHound/packages/javascript/bh-shared-ui/src/utils/exportGraphData.ts

Lines 32 to 38 in 104dfe2

	export const exportToJson = (data: any) => {
	downloadFile({
	data: JSON.stringify(data),
	fileName: 'bh-graph.json',
	fileType: 'text/json',
	});
	};

Additional Information

I can write this feature provided the UX detailed above seems reasonable to the project

[rvazarkar]

Hey @denandz sorry for the late response! That workflow seems appropriate. If you want to do the implementation work, we'd be happy to review your pull request when it comes through. Otherwise our team can take a look at this issue, let me know!

[rvazarkar]

We're going to go ahead and create an internal ticket for this to at least add the epoch to the file name to alleviate some of this issue. If you would like to do a full PR to do the file selector, we would be happy to review that when it comes in