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

Dependency inversion in payload_generator library #9923

Open
busterb opened this issue Apr 24, 2018 · 3 comments
Open

Dependency inversion in payload_generator library #9923

busterb opened this issue Apr 24, 2018 · 3 comments
Assignees
Labels
bug confirmed Issues confirmed by a committer library

Comments

@busterb
Copy link
Member

busterb commented Apr 24, 2018

Following up from #9851, it seems that requiring msf/core/payload_generator should not be required in the middle of lib/msf/util/exe in order to get the required constants, but @jmartin-r7 's comment indicates there is a dependency inversion that is causing it. This ticket is for addressing the core issue.

See #9851 (comment)

@github-actions
Copy link

github-actions bot commented Dec 2, 2020

Hi!

This issue has been left open with no activity for a while now.

We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 30 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open!

As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request.

@github-actions github-actions bot added the Stale Marks an issue as stale, to be closed if no action is taken label Dec 2, 2020
@dwelch-r7 dwelch-r7 added confirmed Issues confirmed by a committer and removed Stale Marks an issue as stale, to be closed if no action is taken labels Dec 2, 2020
@adfoster-r7
Copy link
Contributor

adfoster-r7 commented Apr 26, 2023

@jmartin-r7 Looks like this issue has been around for a while, is this something we want to attic ? 👀

@jmartin-tech
Copy link
Contributor

Looking at the class I still see some scoping concerns, the require that hightlighted the issue has been hidden away by zeitwerk.

I believe there is still work we should do here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug confirmed Issues confirmed by a committer library
Projects
None yet
Development

No branches or pull requests

4 participants