Skip to content

Conversation

ophirharpazg
Copy link
Contributor

@ophirharpazg ophirharpazg commented May 27, 2020

What is this?

Fixes #485 .

Add any further explanations here.

Checklist

  • Have you added an explanation of what your changes do and why you'd like to include them?
  • Have you successfully tested your changes locally?
  • Is the TravisCI build passing?

Proof that it works

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

Changes

Are the commit messages enough? If not, elaborate.

@ophirharpazg ophirharpazg requested a review from ShayNehmad May 27, 2020 16:34
@ophirharpazg ophirharpazg self-assigned this May 27, 2020
@codecov
Copy link

codecov bot commented May 27, 2020

Codecov Report

Merging #670 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #670   +/-   ##
========================================
  Coverage    56.62%   56.62%           
========================================
  Files          120      120           
  Lines         4044     4044           
========================================
  Hits          2290     2290           
  Misses        1754     1754           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d430b91...a07ec92. Read the comment docs.

<i className="glyphicon glyphicon-info-sign" style={{'marginRight': '5px'}}/>
Some Monkey binaries are not found where they should be...<br/>
Try downloading them from <a href="https://github.com/guardicore/monkey/releases/latest" target="blank">here</a>,
at the bottommost section titled "Assets".
Copy link
Contributor

Choose a reason for hiding this comment

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

Ideally we should add "... and place them in monkey/monkey_island/cc/binaries directory."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wish I could make the directory path look like that, I used italics instead 🙄

Copy link
Contributor

Choose a reason for hiding this comment

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

You can do this using import marked from 'marked'; and then styling your text using Markdown. You can see an example of this in monkey_island/cc/ui/src/components/attack/techniques/MitigationsComponent.js.

https://github.com/markedjs/marked

Copy link
Contributor

Choose a reason for hiding this comment

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

Alternatively, use a div with the className="label"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I simply used <code> :)
image

@ShayNehmad ShayNehmad merged commit 33ef1f6 into develop May 31, 2020
@ShayNehmad ShayNehmad deleted the 485/handle-missing-binaries branch May 31, 2020 08:20
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.

If Monkey agent binaries are missing, Monkey Island shows no indication
3 participants