Skip to content
This repository has been archived by the owner on Oct 20, 2023. It is now read-only.

Export fails on campaign with hidden hosts #39

Closed
simplysean opened this issue Nov 11, 2022 · 1 comment · Fixed by #40
Closed

Export fails on campaign with hidden hosts #39

simplysean opened this issue Nov 11, 2022 · 1 comment · Fixed by #40
Labels
bug This issue or pull request addresses broken functionality

Comments

@simplysean
Copy link

Describe the bug
Export of campaign fails if campaign contains hidden hosts.

To Reproduce
Steps to reproduce the behavior:

  1. Create a campaign by importing logs
  2. Hide a host
  3. Export the campaign with option 'Remove all hidden Beacons, Hosts, and Servers' checked
  4. See error

Expected behavior
Campaign to successfully export and the exported campaign does not contain the host(s) that were hidden.

Screenshots
Screen Shot 2022-11-10 at 11 29 41 PM
Screen Shot 2022-11-10 at 11 30 42 PM

Desktop (please complete the following information):

  • OS: macOS Monterey
  • Browser: Chrome
  • Version: 107.0.5304.110

Additional context
I created a campaign, and then hid two hosts (SA-WIN10-21H1-1 and SA-DC01-2019). I then attempted to export the campaign with the option 'Remove all hidden Beacons, Hosts, and Servers' checked, and received the errors in the above screenshots.

@GoldingAustin GoldingAustin added the bug This issue or pull request addresses broken functionality label Nov 11, 2022
GoldingAustin added a commit that referenced this issue Nov 11, 2022
Signed-off-by: GoldingAustin <austin.golding.dev@icloud.com>
@GoldingAustin
Copy link
Collaborator

Thanks for creating a ticket for this! This has been fixed and will be included in the next release tomorrow! It will only work on future campaign uploads; we are working on a simple way to migrate eligible campaigns on version updates and when database schema changes are made.

GoldingAustin added a commit that referenced this issue Nov 11, 2022
Signed-off-by: GoldingAustin <austin.golding.dev@icloud.com>
GoldingAustin added a commit that referenced this issue Nov 11, 2022
This reverts commit e51315f.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue or pull request addresses broken functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants