Skip to content

Conversation

@lloydchang
Copy link
Contributor

@lloydchang lloydchang commented Sep 1, 2024

  • Add success msg, use PascalCase args
  • Add API key reminder
  • Target VM, not localhost, for safety
    • Use 192.168.122.151

Co-authored-by: Haley Lifrieri [email protected]
Co-authored-by: Haley Lifrieri [email protected]
Co-authored-by: Muturi David [email protected]
Co-authored-by: Pardaz Banu Mohammad [email protected]
Co-authored-by: Pardaz Banu Mohammad [email protected]
Co-authored-by: Toluwalope Olateru-Olagbeg [email protected]

* Add success msg, use PascalCase args
* Add API key reminder, simplify with localhost

Co-authored-by: Haley Lifrieri <[email protected]>
Co-authored-by: Haley Lifrieri <[email protected]>
Co-authored-by: Muturi David <[email protected]>
Co-authored-by: Pardaz Banu Mohammad <[email protected]>
Co-authored-by: Pardaz Banu Mohammad <[email protected]>
Co-authored-by: Toluwalope Olateru-Olagbeg <[email protected]>
@lloydchang
Copy link
Contributor Author

lloydchang commented Sep 1, 2024

@lloydchang
Copy link
Contributor Author

@andreashappe While ipa-lab/hackingBuddyGPT#85 auto-assigned to you, this #5 didn't.

@andreashappe
Copy link
Member

please don't target localhost directly. This would mean that a user (that does not know what to do) would hack their own system. We cannot give out instructions like that to end-users.

It is better to use a virtual machine as a target.

lloydchang and others added 2 commits September 2, 2024 09:20
Co-authored-by: Haley Lifrieri <[email protected]>
Co-authored-by: Haley Lifrieri <[email protected]>
Co-authored-by: Muturi David <[email protected]>
Co-authored-by: Pardaz Banu Mohammad <[email protected]>
Co-authored-by: Pardaz Banu Mohammad <[email protected]>
Co-authored-by: Toluwalope Olateru-Olagbeg <[email protected]>
Co-authored-by: Haley Lifrieri <[email protected]>
Co-authored-by: Haley Lifrieri <[email protected]>
Co-authored-by: Muturi David <[email protected]>
Co-authored-by: Pardaz Banu Mohammad <[email protected]>
Co-authored-by: Pardaz Banu Mohammad <[email protected]>
Co-authored-by: Toluwalope Olateru-Olagbeg <[email protected]>
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.

2 participants