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

New profile: singularity (Endgame: Singularity) #6463

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

NetSysFire
Copy link
Contributor

Tested it myself of course. This is a small, python-based game which needs no network access and is generally quite frugal in terms of resources.

I would deem it fine if this is added to firecfg, as there are little to no configurable options and will work out of the box.

etc/profile-a-l/singularity.profile Outdated Show resolved Hide resolved
etc/profile-a-l/singularity.profile Show resolved Hide resolved
etc/profile-a-l/singularity.profile Outdated Show resolved Hide resolved
etc/profile-a-l/singularity.profile Show resolved Hide resolved
@kmk3 kmk3 changed the title New profile: Endgame: Singularity New profile: singularity (Endgame: Singularity) Sep 10, 2024
@kmk3
Copy link
Collaborator

kmk3 commented Sep 13, 2024

@NetSysFire

Disabled more things in the profile.

Does it still work with the changes?

@NetSysFire
Copy link
Contributor Author

This is a small python game that does not need any networking or even external access. I already thought what I did was already a bit overkill as my initial profile for this was even smaller. There is simply no attack surface to justify additional effort in my opinion.

@kmk3
Copy link
Collaborator

kmk3 commented Sep 13, 2024

This is a small python game that does not need any networking or even
external access. I already thought what I did was already a bit overkill as
my initial profile for this was even smaller.

This idea seems backwards.

Why give a program more access than it needs, especially if the standard
restrictions are unlikely to cause problems?

There is simply no attack surface to justify additional effort in my opinion.

What would be the additional effort? I mostly just added the rest of the
common entries from profile.template.

Again, do the changes cause any breakage?

@NetSysFire
Copy link
Contributor Author

I do not have any more motivation to test the new directives. I'd prefer to stick to the things I already tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants