Skip to content

Fix deployment of frontend assets when running with strict umask#98

Merged
ehelms merged 4 commits into
masterfrom
umask
May 7, 2026
Merged

Fix deployment of frontend assets when running with strict umask#98
ehelms merged 4 commits into
masterfrom
umask

Conversation

@evgeni

@evgeni evgeni commented Apr 30, 2026

Copy link
Copy Markdown
Member

On hardened systems umask is often set to something like 0077 instead of the default 0022, which means that group and other will not receive any permission on freshly created files by default.

  • Explicitly set the mode of the apps directory to 0755
  • Set the umask to 0022 when copying assets out of the container

Comment thread manifests/core_host_inventory_frontend.pp
Comment thread lib/puppet/provider/iop_frontend/podman.rb Outdated
evgeni added 2 commits April 30, 2026 08:50
On hardened systems umask is often set to something like `0077` instead
of the default `0022`, which means that `group` and `other` will not
receive any permission on freshly created files by default.

- Explicitly set the mode of the `apps` directory to `0755`
- Set the umask to `0022` when copying assets out of the container
Comment thread lib/puppet/provider/iop_frontend/podman.rb
Comment thread spec/acceptance/core_host_inventory_frontend_spec.rb

@archanaserver archanaserver left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Comment thread manifests/core_host_inventory_frontend.pp
@ehelms ehelms merged commit 9397790 into master May 7, 2026
8 checks passed
@ehelms ehelms deleted the umask branch May 7, 2026 15:25
@archanaserver

Copy link
Copy Markdown

Tested with manual deployment with podman under strict umask (0077). The fix works correctly. The withumask(0022) wrapper and explicit mode 0755 resolve the permission issue on hardened systems.

@evgeni evgeni added the Bug Something isn't working label May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants