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

Add test case to demonstrate the weak masking issue #544

Closed
greg7mdp opened this issue Aug 14, 2024 · 0 comments · Fixed by #622 or #644
Closed

Add test case to demonstrate the weak masking issue #544

greg7mdp opened this issue Aug 14, 2024 · 0 comments · Fixed by #622 or #644
Assignees
@greg7mdp
Labels
👍 lgtm
Milestone
Spring v1.0.0-rc3

Comments

@greg7mdp
Copy link
Contributor

The weak masking issue is described in #534 .

PR #537 implements FSI changes to avoid the issue.

We need a test case which fails on a branch based off commit 7beaa20 , but passes on the main branch after PR #537 was merged.
@enf-ci-bot enf-ci-bot moved this to Todo in Team Backlog Aug 14, 2024
@arhag arhag added 👍 lgtm and removed triage labels Aug 15, 2024
@arhag arhag added this to the Spring v1.0.0 milestone Aug 15, 2024
@greg7mdp greg7mdp moved this from Todo to In Progress in Team Backlog Aug 22, 2024
@greg7mdp greg7mdp mentioned this issue Aug 23, 2024
Merged
@arhag arhag linked a pull request Aug 23, 2024 that will close this issue
[1.0] Add test case to demonstrate the weak masking issue #622
Merged
@greg7mdp greg7mdp mentioned this issue Aug 26, 2024
Merged
@greg7mdp greg7mdp moved this from In Progress to Reviewer Approved in Team Backlog Aug 26, 2024
@github-project-automation github-project-automation bot moved this from Reviewer Approved to Done in Team Backlog Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@greg7mdp greg7mdp

Labels
👍 lgtm
Projects
Team Backlog
Archived in project
Milestone
Spring v1.0.0-rc3
4 participants
@bhazzard @greg7mdp @arhag @enf-ci-bot