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

Testing 19621 #19692

Closed
wants to merge 5 commits into from
Closed

Testing 19621 #19692

wants to merge 5 commits into from

Conversation

dledda-r7
Copy link
Contributor

@dledda-r7 dledda-r7 commented Dec 5, 2024

Note

Can be ignored, only open to walk through testing payload changes as part of #19621

@dledda-r7 dledda-r7 added the payload-testing-branch Runs framework and custom payloads PRs - https://github.com/rapid7/metasploit-framework/pull/19390 label Dec 5, 2024
path: metasploit-framework
ref: ${{ inputs.metasploit_framework_commit }}
ref: fix/met/java/symlinks
Copy link
Contributor

@cgranleese-r7 cgranleese-r7 Dec 5, 2024

Choose a reason for hiding this comment

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

Looks like the Framework commit might be incorrect:

Suggested change
ref: fix/met/java/symlinks
ref: fix/met/java-win-symlink-tests

Copy link
Contributor Author

Choose a reason for hiding this comment

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

need more coffee :/

@@ -185,9 +185,9 @@ jobs:
- name: Checkout metasploit-framework commit
uses: actions/checkout@v4
with:
repository: rapid7/metasploit-framework
repository: zeroSteiner/metasploit-framework
Copy link
Contributor

@cgranleese-r7 cgranleese-r7 Dec 5, 2024

Choose a reason for hiding this comment

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

If we want to test the changes in this PR I believe we would need to point this at your forked repo and not Spencers, as you have branched off of Spencers branch. So we have his commits already, we are now additionally making changes on top of these:

Suggested change
repository: zeroSteiner/metasploit-framework
repository: dledda-r7/metasploit-framework

path: metasploit-framework
ref: ${{ inputs.metasploit_framework_commit }}
ref: fix/met/java-win-symlink-tests
Copy link
Contributor

Choose a reason for hiding this comment

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

and the same with your branch as well:

Suggested change
ref: fix/met/java-win-symlink-tests
ref: testing-19621

@dledda-r7 dledda-r7 closed this Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
payload-testing-branch Runs framework and custom payloads PRs - https://github.com/rapid7/metasploit-framework/pull/19390
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants