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

cannot login even with simplest example #19

Open
nicholasamorim opened this issue May 30, 2019 · 9 comments
Open

cannot login even with simplest example #19

nicholasamorim opened this issue May 30, 2019 · 9 comments

Comments

@nicholasamorim
Copy link

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
Copy link
Author

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

@iwalucas
Copy link

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

@nicholasamorim
Copy link
Author

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

@iwalucas
Copy link

iwalucas commented May 30, 2019

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"]```

@nicholasamorim
Copy link
Author

@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.

@iwalucas
Copy link

maybe we could do a PR to fix this

@nicholasamorim
Copy link
Author

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.

@iwalucas
Copy link

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

@nicholasamorim
Copy link
Author

nicholasamorim commented May 31, 2019

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

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

No branches or pull requests

2 participants