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

WIP: Cloudhsares fixes docs tests #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

danil-topchiy
Copy link

Federated cloud shares fixes, tests, docs

@danil-topchiy
Copy link
Author

Hi @matejak

This branch is based from #9
Having issue with testing federated cloud shares here: I've added new service to docker-compose with one more nextcloud instance to make federated cloudshare with it, but when I send request to nextcloud to make federated cloudshare it responds with
'Sharing Documents failed, could not find admin@app2:80, maybe the server is currently unreachable or uses a self-signed certificate.'
I think because nextcloud cannot resolve app2 domain(even though container by itself does resolve each others domain with ping for example).

Could you, please, have a look at this and may be you know a way how to get around it?

…make cloudshare with, first test (not working yet)
@matejak
Copy link

matejak commented Dec 24, 2018

Sure, the problem that you face is interesting I will take a look into it and probably simplify the docker-compose.yml too while I am at it.

@matejak matejak added the blocked Progress on this item is halted either because of a hard problem, or by missing prerequisite. label Dec 26, 2018
@matejak
Copy link

matejak commented Dec 26, 2018

I am able to reproduce your issue, and I was not able to find a solution. Technically it is possible that the http connection is the problem, but that is somewhat unlikely. While it is possible to debug the issue e.g. by examining outgoing connections on app and incoming connections on app2, it is too much work for a feature that is not a priority for the foreseeable future. Therefore, I propose to leave it open with the blocked label, and focus resources on those other PRs.
Good job nailing the issue though!

@danil-topchiy
Copy link
Author

Agree, thanks for investigating it!

@danil-topchiy danil-topchiy deleted the cloudhsares_fixes_docs_tests branch June 15, 2019 16:22
@danil-topchiy danil-topchiy restored the cloudhsares_fixes_docs_tests branch June 15, 2019 16:24
@danil-topchiy danil-topchiy reopened this Jun 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Progress on this item is halted either because of a hard problem, or by missing prerequisite.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants