Skip to content
This repository has been archived by the owner on Jun 29, 2020. It is now read-only.

Add pax flags to java binary #41

Closed
wants to merge 1 commit into from
Closed

Add pax flags to java binary #41

wants to merge 1 commit into from

Conversation

el-chogo
Copy link

Add pax flags so systems with grsec patchset may run jetty without any further modification (hopefully).

@md5
Copy link
Member

md5 commented Jun 29, 2016

@clinoge Thanks for bringing up this issue, but I wonder if it would make more sense for these flags to be set in the java:*-alpine images instead of here.

Also, whether those flags are set in this image or in java, I'm not sure that we can rely on setfattr in official images yet, since my understanding is that AUFS didn't support extended attributes until quite recently.

If this approach works for your use case, it seems like you could easily call setfattr in an image derived from the jetty image. Others who need PaX support could do the same in their images (hopefully after seeing your approach documented in this PR).

@el-chogo
Copy link
Author

Wow, that's actually true, I should've put that on java:*-alpine. Must've been really sleepy ;).

@md5
Copy link
Member

md5 commented Jun 30, 2016

Closing this request since it needs to happen in the java image if it can be done at all.

@md5 md5 closed this Jun 30, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants