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

Source code leak #26

Open
zhangyandong opened this issue Mar 29, 2023 · 6 comments
Open

Source code leak #26

zhangyandong opened this issue Mar 29, 2023 · 6 comments
Labels
needs reproduction The author did not provide a reproduction repository potential bug

Comments

@zhangyandong
Copy link

This issue seems to have arisen in version 2.3.1
https://github.com/herberttn/bytenode-webpack-plugin/issues/15

@zhangyandong
Copy link
Author

This issue only occurs on Windows

@jjeff
Copy link
Collaborator

jjeff commented Mar 29, 2023

Your link doesn't seem to work for some reason. You're saying that this issue has re-emerged on Windows in the current version (2.3.1)?

I wrote a source code leak test for the electron-bytenode-example project, but I don't think I've updated that since 2.3.1. I'll update that soon and see if I can confirm. It would probably be a good idea to have source code leak tests as part of the tests in this repo too.

@jjeff
Copy link
Collaborator

jjeff commented Mar 30, 2023

I just added some automated testing to the electron-bytenode-example project and I cannot replicate this bug for Windows, Linux, or Mac… at least for an Electron build.

I can't speak for @herberttn, but I'm guessing he's going to need more detail about your issue.

@herberttn
Copy link
Owner

herberttn commented Mar 30, 2023

I can't speak for @herberttn, but I'm guessing he's going to need more detail about your issue.

Absolutely. Right now there are no details, at all.
I'll need a reproduction repository to try and fix it.
@zhangyandong can you make one?

I just added some automated testing to the electron-bytenode-example project and I cannot replicate this bug for Windows, Linux, or Mac… at least for an Electron build.

@jjeff There is a leak test here, added when I fixed #15

@herberttn herberttn added needs reproduction The author did not provide a reproduction repository potential bug labels Mar 30, 2023
@herberttn
Copy link
Owner

@zhangyandong does this still happen? If so, can you create a reproduction repo?

@SamHoque
Copy link

Still seems to happen on a fresh clone of bytenode electron. I tried running the test, the test passes with asar mode on and off. but I can see the string in the code.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs reproduction The author did not provide a reproduction repository potential bug
Projects
None yet
Development

No branches or pull requests

4 participants