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

make apps availble on deSEC #2422

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

make apps availble on deSEC #2422

wants to merge 26 commits into from

Conversation

enoch85
Copy link
Member

@enoch85 enoch85 commented Dec 28, 2022

Signed-off-by: Daniel Hansson [email protected]

Fix #1980

  • Double check that TLS certs are generated for the subdomain. Right now I get this in the logs: cURL error 60: SSL: no alternative certificate subject name matches target host name 'onlyoffice.testingdomainenoch85.dedyn.io' (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://onlyoffice.testingdomainenoch85.dedyn.io/ConvertService.ashx

Nice to have

  • If a specific port is set, also set this port in the docker run command.

Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
@enoch85 enoch85 requested a review from szaimen December 28, 2022 16:19
@enoch85
Copy link
Member Author

enoch85 commented Dec 28, 2022

This is still WIP.

@szaimen
Copy link
Collaborator

szaimen commented Dec 28, 2022

This is still WIP.

Okay

Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
@enoch85
Copy link
Member Author

enoch85 commented Dec 28, 2022

@szaimen Feel free to begin initial tests if you want. 👍

Signed-off-by: Daniel Hansson <[email protected]>
@@ -125,7 +125,7 @@ to finish the setup once this script is done." "$SUBTITLE"
;;&
*"Talk"*)
print_text_in_color "$ICyan" "Downloading the Talk script..."
run_script APP talk
desec_app_tls_menu "DESEC talk_desec" "APP talk"
Copy link
Collaborator

Choose a reason for hiding this comment

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

honestly, I find it a bit strange that this will now always go through this menu. It will be yet another step

Copy link
Member Author

Choose a reason for hiding this comment

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

Do you have a better proposal?

Copy link
Member Author

Choose a reason for hiding this comment

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

ping @szaimen

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we maybe check for the dessec folder and only show that menu if dessec is already configured?

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm, should work.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thought about this, what about users that doesn't have deSEC installed and wants to use it?

lib.sh Outdated Show resolved Hide resolved
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
@enoch85
Copy link
Member Author

enoch85 commented Dec 29, 2022

Talk works out of the box.

Onlyoffice gets 504 timeout, but everything is generated as it should.

Collabora is OK when hitting "Save" in the UI, but no document is loaded when opening it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement deSEC in Collabora | OnlyOffice docker versions
2 participants