Skip to content

OpenSSF gold badge: apply hardening fixes from #1241 without enabling hardening itself#1348

Merged
TonyXiang8787 merged 5 commits into
mainfrom
pgm/feature/hardening-fixes
Mar 31, 2026
Merged

OpenSSF gold badge: apply hardening fixes from #1241 without enabling hardening itself#1348
TonyXiang8787 merged 5 commits into
mainfrom
pgm/feature/hardening-fixes

Conversation

@mgovers
Copy link
Copy Markdown
Member

@mgovers mgovers commented Mar 31, 2026

Relates to #1241 .

In #1241 , a number of fixes were applied. This PR cherry-picks those fixes without actually enabling hardening itself. This enables safer code without enforcing things that are still TBD.

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers self-assigned this Mar 31, 2026
@mgovers mgovers added the improvement Improvement on internal implementation label Mar 31, 2026
@mgovers mgovers enabled auto-merge March 31, 2026 07:39
figueroa1395
figueroa1395 previously approved these changes Mar 31, 2026
Copy link
Copy Markdown
Member

@figueroa1395 figueroa1395 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you rename the PR? We already have the silver badge, perhaps we can name name this one gold badge, since this is needed for that milestone.

Besides this, no more comments from me. LGTM.

@mgovers mgovers changed the title OpenSSF silver badge: apply hardening fixes from #1241 without enabling hardening itself OpenSSF gold badge: apply hardening fixes from #1241 without enabling hardening itself Mar 31, 2026
mgovers added 2 commits March 31, 2026 11:20
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@sonarqubecloud
Copy link
Copy Markdown

@TonyXiang8787 TonyXiang8787 disabled auto-merge March 31, 2026 10:28
@TonyXiang8787
Copy link
Copy Markdown
Member

Hi @mgovers, @figueroa1395,

I took out this PR out of merge queue because I still have some questions.

We have indeed some decisions to make about whether and how enable hardening build flags for release distribution.

But we can enable harnding flags in development cmake presets right?. That should check in the CI?

@mgovers
Copy link
Copy Markdown
Member Author

mgovers commented Mar 31, 2026

But we can enable harnding flags in development cmake presets right?. That should check in the CI?

I think so, but i'd consider that a follow-up. #1241 has been open for so long that the fixes (which are actual cases of UB) should not be delayed any longer. IMO, we should scope out the other choices to a separate PR (maybe we can decide on the exact form on thursday)

@TonyXiang8787 TonyXiang8787 enabled auto-merge March 31, 2026 10:32
@TonyXiang8787 TonyXiang8787 added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit 3c0b59b Mar 31, 2026
31 checks passed
@TonyXiang8787 TonyXiang8787 deleted the pgm/feature/hardening-fixes branch March 31, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement on internal implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants