OpenAS2 / OpenAs2App

OpenAS2 is a java-based implementation of the EDIINT AS2 standard. It is intended to be used as a server. It is extremely configurable and supports a wide variety of signing and encryption algorithms.
https://sourceforge.net/projects/openas2/
GNU General Public License v3.0
185 stars 137 forks source link

Dockerfile fix #226

Closed kek-Sec closed 3 years ago

kek-Sec commented 3 years ago

Current dockerfile produces the following error:/bin/sh: 1: /opt/openas2//bin/start-container.sh: not found

Solution, change entry path to: ENTRYPOINT $OPENAS2_BASEbin/start-container.sh

This way you don't have the double // on bin

igwtech commented 3 years ago

It would be better to remove the / from the ENV variable, that proposed solution will try expand the ENV variable "OPENAS2_BASEBIN"

On Fri, Jul 23, 2021 at 5:56 AM kek-Sec @.***> wrote:

Current dockerfile produces the following error: /bin/sh: 1: /opt/openas2//bin/start-container.sh: not found

Solution, change entry path to: ENTRYPOINT $OPENAS2_BASEbin/start-container.sh

This way you don't have the double // on bin

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/OpenAS2/OpenAs2App/issues/226, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2QNDZDYQJEXWBMHZPDPILTZE4DXANCNFSM5A3WCQWQ .

-- Javier Muñoz

CEO

Ingeniería | Greicodex Software C.A. (+58) 212-7629120 <(+58)+212-7629120> @.*** greicodex.com Av. Francisco Solano con Calle La Iglesia. Torre Centro Solano Plaza I. Piso 9, Oficina PH-A., Urb. Sabana Grande. Municipio Libertador. Dtto. Capital. Código postal 1050.

[image: facebook] http://facebook.com/greicodex [image: twitter] https://twitter.com/greicodex

kek-Sec commented 3 years ago

The error remains the same after the variable change , it must be a problem with the copying of the workdir the start-container.sh script is not contained

mackieee commented 3 years ago

+1 Came across this today on a build, had changed the ENV variable as @igwtech had suggested - yielded the same response.

LeoWare commented 3 years ago

Hi. I've made some corrections and submitted a pull request.

229

kek-Sec commented 3 years ago

@LeoWare thanks for your time