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.

Sign up for GitHub

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

Jump to bottom

Resulting docker image is incompatible with eliptic curves webservers(outdated libpoco) #895

Closed
kratzersmz opened this issue Dec 5, 2020 · 1 comment
Closed

Resulting docker image is incompatible with eliptic curves webservers(outdated libpoco) #895

kratzersmz opened this issue Dec 5, 2020 · 1 comment
Labels
bug Something isn't working docker Issues with containers

Comments

@kratzersmz
Copy link

Describe the bug
https://www.collaboraoffice.com/online/collabora-online-update-4-2-1-available/ -->
"On CentOS 8, Ubuntu 18.04, and Debian 10, which have OpenSSL 1.1.1, now we link loolwsd with Poco 1.10.1. That solves SSL connection issues, e.g. connecting to sites that have certificates made with elliptic curve key"

The current provided docker build stuff will not result in docker images which are compatible with eliptic curves:
https://github.com/CollaboraOnline/online/tree/master/docker ---> still outdated libpoco
https://github.com/CollaboraOnline/online/tree/libreoffice-7-0/docker ---> still outdated libpoco

With outdated libpoco i mean that libpoco has problems with eliptic curves...

To Reproduce
https://github.com/CollaboraOnline/online/tree/libreoffice-7-0/docker

Expected behavior
Connect to e.g. an nextcloud instance with eliptic curves enabled should work without problems...

Actual behavior
lool cant connect to nextcloud storage with ssl error
@kratzersmz kratzersmz added bug Something isn't working unconfirmed labels Dec 5, 2020
@timar
Copy link
Member

timar commented Jan 11, 2021

I'll fix this on master, but libreoffice-7-0 is out of scope.

@timar timar added docker Issues with containers and removed unconfirmed labels Jan 11, 2021
timar added a commit that referenced this issue Jan 11, 2021
@timar
Fix #895: build poco from source, link poco statically, add openSUSE …
70ff516 
…target, and other fixes

Signed-off-by: Andras Timar <andras.timar@collabora.com>
Change-Id: I525ee004771ad335e41e4602e044d4fd8d8a19bc
@timar timar closed this as completed in 289a51d Jan 11, 2021
@hcvcastro hcvcastro mentioned this issue Jan 18, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working docker Issues with containers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

gh pr 17 hcvcastro/online
2 participants
@timar @kratzersmz