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

Alpine name collision prevents adding ability to an adversary #2350

Closed
cyber-arsenull opened this issue Nov 5, 2021 · 9 comments · Fixed by #2351
Closed

Alpine name collision prevents adding ability to an adversary #2350

cyber-arsenull opened this issue Nov 5, 2021 · 9 comments · Fixed by #2351
Assignees
Labels

Comments

@cyber-arsenull
Copy link
Contributor

Describe the bug
As mentioned in part of #2348 you currently cannot add an ability to an adversary.
This is due to the naming collision of showAbilityModal in profiles.html and abilities.html. Seems the abilities one always wins this fight, and you end up with the incorrect modal.

To Reproduce
Steps to reproduce the behavior:

  1. Click abilities
  2. Click + Create an Ability
  3. Fill out the needed information before + Add Cleanup Command
  4. Click + Add Cleanup Command and the window will be closed immediately, as well as all the tabs already opened

Expected behavior
An ability selection screen (modal) should appear from profiles.html.

Additional context
Proof of issue's cause hypothesis: If you change the name of either the issue is resolved.

@github-actions
Copy link

github-actions bot commented Nov 5, 2021

Looks like your first issue -- we aim to respond to issues as quickly as possible. In the meantime, check out our documentation here: http://caldera.readthedocs.io/

@radioheado
Copy link

Thank you so much for the quick update!

@cyber-arsenull
Copy link
Contributor Author

cyber-arsenull commented Nov 5, 2021

No problem! Thanks for the good bug reports.
Seems like even prior to the naming collision, the profiles.html showAbilityModal was overwritten with the one from abilities.html
@argaudreau
I made a PR that reverts the profiles.html change, and changes the name in profiles.html to showAbilityChoiceModal:
ff49573

I'm not sure if there were changes in the profiles.html commit that are separate from this issue and should stay in:
13a246b

@cyber-arsenull cyber-arsenull self-assigned this Nov 5, 2021
@clr2of8
Copy link

clr2of8 commented Nov 8, 2021

Just installed master today 11/8/21 and getting this same error

image

@clr2of8
Copy link

clr2of8 commented Nov 8, 2021

any work arounds?

@cyber-arsenull
Copy link
Contributor Author

@clr2of8 hey, try this branch: bugfix/add_adversary_ability for the related PR to this issue #2351
Feel free to comment if this does not work, or any other issues arise related to the PR solution.

@clr2of8
Copy link

clr2of8 commented Nov 9, 2021

Thanks @cyber-arsenull, I gave that a try and while it did fix the problem at hand, a new problem presented itself wherein I can't save the Adversary after adding any ability. I get "! Error saving profile" whenever I click the green "Save Profile" button. In fact, if I just load the Adversary and remove one of the abilities already in it, I get the same error saving.

image

@clr2of8
Copy link

clr2of8 commented Nov 9, 2021

OK, nevermind, it seemed to be a problem with only my custom built "Certifiable Adversary", I deleted and recreated and now it is saving.

@clr2of8
Copy link

clr2of8 commented Nov 9, 2021

Don't know if this is a problem with your branch specifically, but the "create ability" module doesn't get marked complete after adding the "My test ability" ability to the "Certifiable" adversary. I started the training over completely and then found that the "Contact points" module won't complete either, tried to deploy sandcat as http protocol on remote windows vm but it says malformed request comes into the server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants