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

Update manual run page #3328

Merged
merged 4 commits into from
May 10, 2023
Merged

Update manual run page #3328

merged 4 commits into from
May 10, 2023

Conversation

shreyamalviya
Copy link
Contributor

What does this PR do?

Fixes #2593

PR Checklist

  • Have you added an explanation of what your changes do and why you'd like to include them?
  • Is the TravisCI build passing?
  • Was the CHANGELOG.md updated to reflect the changes?
  • Was the documentation framework updated to reflect the changes?
  • Have you checked that you haven't introduced any duplicate code?

Testing Checklist

  • Added relevant unit tests?
  • Do all unit tests pass?
  • Do all end-to-end tests pass?
  • Any other testing performed?

    Tested by running the Island

  • If applicable, add screenshots or log transcripts of the feature working

image

Copy link
Contributor

@ilija-lazoroski ilija-lazoroski left a comment

Choose a reason for hiding this comment

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

I think this issue was primarily to add the list of IPs as list of servers to the command.

@mssalvatore
Copy link
Collaborator

I think this issue was primarily to add the list of IPs as list of servers to the command.

+1

These changes are good. The clarification is needed. But we want to have all Island IPs in the agent command's server list.

@shreyamalviya
Copy link
Contributor Author

image

CHANGELOG.md Outdated
@@ -19,6 +19,7 @@ Changelog](https://keepachangelog.com/en/1.0.0/).
- Hard-coded Mimikatz credentials collector to a plugin. #3168
- Hard-coded Zerologon exploiter to a plugin. #3164
- Hard-coded SSH credentials collector to a plugin. #3169
- Manual run page UI. #3328
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use the issue number, not the PR number.

@mssalvatore mssalvatore merged commit 774953d into develop May 10, 2023
@mssalvatore mssalvatore deleted the 2593-update-manual-run-page branch May 10, 2023 14:45
mssalvatore added a commit that referenced this pull request Mar 29, 2024
We ran into a problem with #2593 where it was improperly resolved.
PR #3328 did not fully resolve the issue, but was merged because it made
necessary improvements. However, because the PR template contained
"Fixes #...", when PR #3328 was merged GitHub automatically closed
Issue #2593. This flew under the radar and Issue #2593 was therefore not
properly resolved until 2 releases later.

By changing this template, we hope to avoid similar sequences of events
in the future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update manual run page
3 participants