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.

Sign up for GitHub

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

Jump to bottom

Use target remote for remote connections #1151

Open
wants to merge 42 commits into
base: main
Choose a base branch
from
Open

Use target remote for remote connections #1151

wants to merge 42 commits into from

Conversation

hugsy
Copy link
Owner

@hugsy hugsy commented Nov 3, 2024 •
edited
Loading

Description

This PR obsoletes gef-remote to be completely replaced by pre and post hooks on target remote directly.
We use some heuristics to determine what type of remote we're connected to reliably.
In some old qemu, we still have to mock a memory layout, but the whole sync thing is not necessary any longer.

Fixes #1080
Fixes #1128

Checklist

  • My code follows the code style of this project.
  • My change includes a change to the documentation, if required.
  • If my change adds new code, adequate tests have been added.
  • I have read and agree to the CONTRIBUTING document.

@hugsy hugsy mentioned this pull request Nov 3, 2024
Open
8 tasks
@hugsy hugsy self-assigned this Nov 3, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 3, 2024

🤖 Coverage update for a277661 🔴

Old New
Commit 9fb1d71 a277661
Score 71.2662% 71.197% (-0.0692)

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 4, 2024

🤖 Coverage update for 6d166f0 🔴

Old New
Commit 9fb1d71 6d166f0
Score 71.2662% 71.197% (-0.0692)

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 5, 2024

🤖 Coverage update for 79447be 🔴

Old New
Commit 9fb1d71 79447be
Score 71.2662% 71.197% (-0.0692)

@hugsy hugsy linked an issue Nov 5, 2024 that may be closed by this pull request
Implement pre/post hooks on target remote #1150
Open
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2024

🤖 Coverage update for ea873bb 🔴

Old New
Commit 9fb1d71 ea873bb
Score 71.2662% 71.197% (-0.0692)

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2024

🤖 Coverage update for 4ae683f 🔴

Old New
Commit 9fb1d71 4ae683f
Score 71.2662% 71.197% (-0.0692)

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2024

🤖 Coverage update for 8130895 🔴

Old New
Commit 9fb1d71 8130895
Score 71.2662% 71.2358% (-0.0304)

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2024

🤖 Coverage update for 7973a3f 🔴

Old New
Commit 9fb1d71 7973a3f
Score 71.2662% 71.197% (-0.0692)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 03f9cf9 🟢

Old New
Commit 0f21eea 03f9cf9
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 2a5c585 🟢

Old New
Commit 0f21eea 2a5c585
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 36c953c 🟢

Old New
Commit 0f21eea 36c953c
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 68505a6 🟢

Old New
Commit 0f21eea 68505a6
Score 71.2413% 71.2607% (0.0194)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 02af4d4 🟢

Old New
Commit 0f21eea 02af4d4
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for c89c280 🟢

Old New
Commit 0f21eea c89c280
Score 71.2413% 71.2607% (0.0194)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 77aa954 🟢

Old New
Commit 0f21eea 77aa954
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 623ebf1 🟢

Old New
Commit 0f21eea 623ebf1
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 532e45d 🟢

Old New
Commit 0f21eea 532e45d
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for fd0bf0a 🟢

Old New
Commit 0f21eea fd0bf0a
Score 71.2413% 71.2413% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for d262ca5 🟢

Old New
Commit 9f2737a d262ca5
Score 71.1786% 71.1786% (0)

@hugsy hugsy mentioned this pull request Nov 11, 2024
Open
9 tasks
@github-actions GitHub Actions
Copy link

🤖 Coverage update for 7a9e494 🟢

Old New
Commit 9f2737a 7a9e494
Score 71.1786% 71.1786% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for b9f564f 🟢

Old New
Commit 9f2737a b9f564f
Score 71.1786% 71.1786% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 7ea8588 🟢

Old New
Commit 9f2737a 7ea8588
Score 71.1786% 71.1786% (0)

@github-actions GitHub Actions
Copy link

🤖 Coverage update for 7ea8588 🟢

Old New
Commit 9f2737a 7ea8588
Score 71.1786% 71.2367% (0.0581)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ValekoZ ValekoZ ValekoZ left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@hugsy hugsy

Labels
api-change enhancement feature:remote
Projects
None yet
Milestone
next
Development

Successfully merging this pull request may close these issues.

Implement pre/post hooks on target remote [Bug] 'gef-remote' failed to execute properly, reason: [Errno 17] File exists Possible GEF dereference issue with gef-remote
2 participants
@hugsy @ValekoZ