Skip to content

Commit

Permalink
Merge pull request #8 from TheGuyDanish/master
Browse files Browse the repository at this point in the history 
Use Chromium and reduce Java 7 security, aligned with ARM
  • Loading branch information
@TokugawaHeavyIndustries
TokugawaHeavyIndustries authored Dec 7, 2020
2 parents 68a237a + 36fa7eb commit 22618ea
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
7 changes: 3 additions & 4 deletions Dockerfile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,10 @@ FROM ubuntu:14.04
MAINTAINER Kyle Anderson <[email protected]>

ENV DEBIAN_FRONTEND noninteractive
RUN apt-get -y install software-properties-common
RUN add-apt-repository ppa:jonathonf/firefox-esr
RUN apt-get update && apt-get -y install xvfb x11vnc wget \
supervisor fluxbox icedtea-7-plugin net-tools \
python-numpy firefox-esr=52.9.0esr-1~14.04.york0
supervisor fluxbox icedtea-7-plugin net-tools python-numpy \
chromium-browser
RUN sed -e '/^jdk.jar.disabledAlgorithms/s/^/#/' -i /usr/lib/jvm/java-7-openjdk-amd64/jre/lib/security/java.security
ADD supervisord.conf /etc/supervisor/conf.d/supervisord.conf

WORKDIR /root/
Expand Down
4 changes: 2 additions & 2 deletions supervisord.conf
View file
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ autorestart=true
command=/usr/bin/fluxbox
autorestart=true

[program:firefox]
command=/usr/bin/firefox
[program:chromium]
command=/usr/bin/chromium-browser --no-sandbox
autorestart=true


3 comments on commit 22618ea

@lmacka
Copy link

@lmacka lmacka commented on 22618ea Jan 5, 2021

Choose a reason for hiding this comment

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

Just a heads up, switching to Chromium has broken this on HP ILO100 cards. Still works fine with the previous build though.

@solarkennedy
Copy link
Owner

Choose a reason for hiding this comment

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

Hmm. Please make a PR for whatever you need.
We don't have to start up every browser, but we could still include firefox?

@TokugawaHeavyIndustries
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@lmacka Just opened #9 for this. If you could post any relevant info over there for me, that'd be great!

Please sign in to comment.