docker-cronicle icon indicating copy to clipboard operation
docker-cronicle copied to clipboard

cannot login even with simplest example

Open nicholasamorim opened this issue 6 years ago • 9 comments

Running this

docker run --name cronicle --hostname localhost -p 3012:3012 intelliops/cronicle:latest

Boots up the image but the browser cannot login. We see the "Waiting for master server" for some time and then the frame just looks white.

image

nicholasamorim avatar May 30 '19 09:05 nicholasamorim

I've bumped the version to 0.8.29 hoping that this would be related but the behaviour above still happens.

nicholasamorim avatar May 30 '19 13:05 nicholasamorim

I made my own dockerfile and its working now... @nicholasamorim let me know if u want it

iwalucas avatar May 30 '19 13:05 iwalucas

@iwalucas Can you paste here? Even if only to help diagnosing this dockerfile

nicholasamorim avatar May 30 '19 13:05 nicholasamorim

the entrypoint.sh is the same....

FROM node:alpine

RUN apk add --no-cache git curl wget perl bash perl-pathtools tar \
             procps tini
			 
RUN curl -s https://raw.githubusercontent.com/jhuckaby/Cronicle/master/bin/install.js | node


COPY        entrypoint.sh /entrypoint.sh

EXPOSE     3012

# data volume is also configured in entrypoint.sh
VOLUME     ["/opt/cronicle/data", "/opt/cronicle/logs", "/opt/cronicle/plugins"]


CMD        ["./entrypoint.sh"]```

iwalucas avatar May 30 '19 13:05 iwalucas

@iwalucas Yours work, yes.

This seems to be the offending line on this dockerfile:

ENV        CRONICLE_socket_io_transports '["polling", "websocket"]'

If this line is removed, it works. Making it be empty or ["websocket"] only doesn't work neither.

nicholasamorim avatar May 30 '19 14:05 nicholasamorim

maybe we could do a PR to fix this

iwalucas avatar May 30 '19 14:05 iwalucas

I would say so yes. Cronicle's doc states about those two values together: "please only do this if you know exactly what you are doing, and why".

I'd still like to hear @belsander take on the matter, though.

nicholasamorim avatar May 30 '19 14:05 nicholasamorim

I forked this repo on my account.... I needed to install python3 anyway so will wait for @belsander to let us know what the problem is about so I can have a better setup

iwalucas avatar May 30 '19 19:05 iwalucas

Cronicle project actually states they do not accept arrays as envvars:

Almost every configuration property can be overridden using this environment variable syntax. The only exceptions are things like arrays, e.g. log_columns and socket_io_transports.

I have an image that fixes this: https://hub.docker.com/repository/docker/nicholasamorim/cronicle

nicholasamorim avatar May 31 '19 06:05 nicholasamorim