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

realContentHash replace content in user code #5339

Closed
ahabhgk opened this issue Jan 16, 2024 · 5 comments
Closed

realContentHash replace content in user code #5339

ahabhgk opened this issue Jan 16, 2024 · 5 comments

Comments

@ahabhgk
Copy link
Contributor

ahabhgk commented Jan 16, 2024

Also see webpack/webpack#14058

optimization.realContentHash will use RegExp to replace the contenthash in generated code, it's possible to replace the user code which contains the same content as contenthash. Although the longer the contenthash length, the less likely it is to occur, it is still easy to appear when the contenthash length is very short.

If you want to implement this, we encourage you also contribute this back to webpack.

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Jan 16, 2024
@ahabhgk ahabhgk added gsoc-2024 and removed team The issue/pr is created by the member of Rspack. labels Jan 16, 2024
@romanlex
Copy link

I can confirm issue from webpack
image

Copy link

stale bot commented Jun 16, 2024

This issue has been automatically marked as stale because it has not had recent activity. If this issue is still affecting you, please leave any comment (for example, "bump"). We are sorry that we haven't been able to prioritize it yet. If you have any new additional information, please include it with your comment!

@stale stale bot added the stale label Jun 16, 2024
@hardfist
Copy link
Contributor

bump

@stale stale bot removed the stale label Aug 13, 2024
Copy link

stale bot commented Nov 4, 2024

This issue has been automatically marked as stale because it has not had recent activity. If this issue is still affecting you, please leave any comment (for example, "bump"). We are sorry that we haven't been able to prioritize it yet. If you have any new additional information, please include it with your comment!

@chenjiahan
Copy link
Member

I will close this and let's track the progress here: #8474

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants