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

Add a build of the Dockerfile in github action #115

Closed
fwininger opened this issue Apr 25, 2021 · 2 comments · Fixed by #120
Closed

Add a build of the Dockerfile in github action #115

fwininger opened this issue Apr 25, 2021 · 2 comments · Fixed by #120

Comments

@fwininger
Copy link
Contributor

The master branch are not able to build the projet Dockerfile with de brower-cookie3 dependency.

Please add a build of the Dockerfile in github action.

@devl00p
Copy link
Contributor

devl00p commented Apr 25, 2021

It seems related to pyca/cryptography#5753

I managed to get the image working with the following dockerfile (just ading python3-pip to packages to install using apt):

FROM debian:buster-slim as build

ENV DEBIAN_FRONTEND=noninteractive \
  LANG=en_US.UTF-8

WORKDIR /usr/src/app

RUN apt update
RUN apt install python3 python3-setuptools python3-pip ca-certificates -y \
  && apt clean -yq \
  && apt autoremove -yq \
  && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* \
  && truncate -s 0 /var/log/*log

COPY . .

RUN python3 setup.py install

FROM debian:buster-slim

ENV DEBIAN_FRONTEND=noninteractive \
  LANG=en_US.UTF-8 \
  PYTHONDONTWRITEBYTECODE=1

RUN apt update
RUN apt install python3 python3-setuptools python3-pip -y \
  && apt clean -yq \
  && apt autoremove -yq \
  && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* \
  && truncate -s 0 /var/log/*log

COPY --from=build /usr/local/lib/python3.7/dist-packages/ /usr/local/lib/python3.7/dist-packages/
COPY --from=build /usr/local/bin/wapiti /usr/local/bin/wapiti-getcookie /usr/local/bin/

CMD ["wapiti"]

@fwininger
Copy link
Contributor Author

close with #120

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 a pull request may close this issue.

2 participants