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

Fix travis #42

Closed
wants to merge 5 commits into from
Closed

Fix travis #42

wants to merge 5 commits into from

Conversation

tonsV2
Copy link
Contributor

@tonsV2 tonsV2 commented Sep 5, 2020

No description provided.

@tonsV2 tonsV2 mentioned this pull request Sep 5, 2020
@sobolevn
Copy link
Member

sobolevn commented Sep 5, 2020

Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@tonsV2
Copy link
Contributor Author

tonsV2 commented Sep 5, 2020

Locally it works just fine, port 2015 returns 404. On travis it returns 400. I don't know why.

@sobolevn
Copy link
Member

sobolevn commented Sep 5, 2020

@tonsV2 maybe we can have a look at the response headers and body, not only status code? We can also enable logs on caddy's side to figure this out. It might be a bug!

@tonsV2 tonsV2 force-pushed the master branch 2 times, most recently from 1742011 to c0e3063 Compare September 6, 2020 03:21
@tonsV2
Copy link
Contributor Author

tonsV2 commented Sep 6, 2020

Yeah, Ideally we should assert all open ports. I've added the verbose flag to the curl command but it gives little information. I've tried to enable debugging on Caddy but I can't seem to get it working. Granted I haven't put a lot of effort into it. But do you have any idea about how to do it?

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 this pull request may close these issues.

2 participants