Skip to content

Remove world writable from files in OOD packaged gems#4717

Merged
treydock merged 5 commits intomasterfrom
rm-world-write
Oct 27, 2025
Merged

Remove world writable from files in OOD packaged gems#4717
treydock merged 5 commits intomasterfrom
rm-world-write

Conversation

@treydock
Copy link
Contributor

Fixes #4712

@github-project-automation github-project-automation bot moved this to Awaiting Review in PR Review Pipeline Oct 23, 2025
@treydock treydock force-pushed the rm-world-write branch 2 times, most recently from c1ce914 to d543bc1 Compare October 23, 2025 19:51
@treydock treydock changed the title Remove world writable files from OOD packaged gems Remove world writable from files in OOD packaged gems Oct 23, 2025
@treydock treydock force-pushed the rm-world-write branch 2 times, most recently from 026d6bc to c966c64 Compare October 24, 2025 14:05
@treydock treydock requested a review from johrstrom October 24, 2025 14:43
@treydock
Copy link
Contributor Author

I think we'll want to backport this into 4.0 and 3.1 , at least 4.0.

@johrstrom johrstrom closed this Oct 24, 2025
@johrstrom johrstrom reopened this Oct 24, 2025
@github-project-automation github-project-automation bot moved this from Awaiting Review to Merged/Closed in PR Review Pipeline Oct 24, 2025
@github-project-automation github-project-automation bot moved this from Merged/Closed to Awaiting Review in PR Review Pipeline Oct 24, 2025
Copy link
Contributor

@johrstrom johrstrom left a comment

Choose a reason for hiding this comment

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

LGTM

@treydock treydock merged commit 47934bf into master Oct 27, 2025
43 of 44 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Review to Merged/Closed in PR Review Pipeline Oct 27, 2025
@treydock treydock deleted the rm-world-write branch October 27, 2025 16:39
treydock added a commit that referenced this pull request Oct 27, 2025
* Remove world writable files from OOD packaged gems
Fixes #4712

* Also remove global write for deb packages

* Add debugging

* Exclude links from the check

* Move debugging output into actual check
treydock added a commit that referenced this pull request Oct 27, 2025
* Remove world writable files from OOD packaged gems
Fixes #4712

* Also remove global write for deb packages

* Add debugging

* Exclude links from the check

* Move debugging output into actual check
harshit-soora pushed a commit to harshit-soora/ondemand that referenced this pull request Oct 31, 2025
* Remove world writable files from OOD packaged gems
Fixes OSC#4712

* Also remove global write for deb packages

* Add debugging

* Exclude links from the check

* Move debugging output into actual check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

ondemand-gems-4.0.7-1-4.0.7-1 RPM creating world writeable directories.

2 participants