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.

Sign up for GitHub

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

Jump to bottom

praktomat.policy vs java.policy #307

Open
chris21k opened this issue Dec 17, 2019 · 3 comments · Fixed by hso-praktomat/praktomat#19 · May be fixed by #363
Open

praktomat.policy vs java.policy #307

chris21k opened this issue Dec 17, 2019 · 3 comments · Fixed by hso-praktomat/praktomat#19 · May be fixed by #363

Comments

@chris21k
Copy link

Praktomat/src/checker/checker/DejaGnu.py line 120 references not existing file "praktomat.policy" instead of in directory Praktomat/src/checker/scripts/ supplied "java.policy". Thus, 'spawn java' starts a jvm with empty policies via Praktomat/src/checker/scripts/java. A workaround without touching the python code is
$ ln -s java.policy praktomat.policy
in this directory.
@ifrh
Copy link
Contributor

ifrh commented Nov 25, 2021 •
edited
Loading

Just found on https://docs.oracle.com/en/java/javase/17/security/java-security-overview1.html#GUID-BBEC2DC8-BA00-42B1-B52A-A49488FCF8FE

The Security Manager and APIs related to it have been deprecated and are subject to removal in a future release. There is no replacement for the Security Manager. See JEP 411 - https://openjdk.java.net/jeps/411 - for discussion and alternatives.

corresponding information here:
https://bugs.openjdk.java.net/browse/JDK-8264713

@ifrh
Copy link
Contributor

ifrh commented Jan 28, 2022

@chris21k could you please update the issue headline, that one can read, that Security Manager is deprecated?

referencing to an old mail to praktomat mailinglist (8.10.2016 of @nomeata ), perhaps firejail https://firejail.wordpress.com/ could use as sandbox i.e. on systems where Docker containers are "too big".

@ifrh
Copy link
Contributor

ifrh commented Aug 10, 2022

Implementation changed to

Praktomat/src/checker/checker/DejaGnu.py

Line 122 in 92d1cf5

environ['POLICY'] = join(script_dir, "java.policy")

via commit bc585cf#diff-e64c26e50d196ca3f4cccef7ec36c569a692481cafc1b02a789b44f321e7e590 merged to master

@ifrh ifrh mentioned this issue Aug 10, 2022
Open
ifrh pushed a commit to ifrh/Praktomat that referenced this issue Jan 23, 2023
Remove Java's Security Manager deprecation message from Checker-Outputs.
b7101a2 
add some documentation information into java.policy and junit.policy
see KITPraktomatTeam#350
see KITPraktomatTeam#307
closes KITPraktomatTeam#307
This was referenced Jan 23, 2023
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@ifrh @chris21k