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

feat(wasm-builder): remap paths to hide username in debug builds #2865

Merged
merged 3 commits into from
Jun 28, 2023

Conversation

StackOverflowExcept1on
Copy link
Member

@StackOverflowExcept1on StackOverflowExcept1on commented Jun 26, 2023

Resolves #2812

The path remapping will be like this. This is useful to understand if the panic was in dependencies or in project.

  • /home/user -> /home
  • /path/to/smart-contract -> /code
  • /home/user/.cargo -> /cargo
  • /home/user/.cargo/git/checkouts/ -> /deps

@gear-tech/dev

@StackOverflowExcept1on StackOverflowExcept1on added A0-pleasereview PR is ready to be reviewed by the team D5-tooling Helper tools and utilities labels Jun 26, 2023
utils/wasm-builder/src/lib.rs Outdated Show resolved Hide resolved
utils/wasm-builder/src/lib.rs Show resolved Hide resolved
@StackOverflowExcept1on StackOverflowExcept1on added A0-pleasereview PR is ready to be reviewed by the team and removed A0-pleasereview PR is ready to be reviewed by the team labels Jun 27, 2023
utils/wasm-builder/src/lib.rs Outdated Show resolved Hide resolved
@breathx breathx added the A2-mergeoncegreen PR is ready to merge after CI passes label Jun 28, 2023
@StackOverflowExcept1on StackOverflowExcept1on merged commit d4a6248 into master Jun 28, 2023
10 checks passed
@StackOverflowExcept1on StackOverflowExcept1on deleted the av/issue-2812 branch June 28, 2023 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-pleasereview PR is ready to be reviewed by the team A2-mergeoncegreen PR is ready to merge after CI passes D5-tooling Helper tools and utilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider to remove username (full path) from panic message
3 participants