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

Config pitfalls after v4 upgrade #7176

Open
Tracked by #7370
xoxys opened this issue Aug 31, 2023 · 9 comments
Open
Tracked by #7370

Config pitfalls after v4 upgrade #7176

xoxys opened this issue Aug 31, 2023 · 9 comments

Comments

@xoxys
Copy link
Contributor

xoxys commented Aug 31, 2023

Describe the bug

  • OCIS_LDAP_BIND_PASSWORD not working and must use LDAP_BIND_PASSWORD while all other LDAP env vars were prefixed with OCIS_
  • OCIS_LDAP_USER_ENABLED_ATTRIBUTE and OCIS_LDAP_DISABLE_USER_MECHANISM not documented for v4 and results in an ugly breaking change while using an external LDAP
  • APP_PROVIDER_SERVICE_NAME and APP_PROVIDER_EXTERNAL_ADDR breaks office integrations and are not part of the upgrade guide
  • APP_PROVIDER_SERVICE_NAME=ocis-provider-collabora and APP_PROVIDER_EXTERNAL_ADDR=com.owncloud.api.ocis-provider-collabora need to match which is not documented and should be handled automatically
  • MICRO_REGISTRY is not mentioned in the upgrade guide
  • MICRO_REGISTRY env var is not namespaced or prefixed with OCIS_
  • documented deprecated env vars for v4 are not accurate (see above)/helpful

image

@xoxys xoxys added the Type:Bug label Aug 31, 2023
@micbar
Copy link
Contributor

micbar commented Sep 1, 2023

@ScharfViktor Could we just run an new upgrade from 3.0.0 to 4.0.0 with the WOPI example and docuement the step-by-step process.

@dragonchaser We should prefix that LDAP_BIND_PASSWORD with ocis and include it in the 4.0.1 patch release.

@micbar
Copy link
Contributor

micbar commented Sep 1, 2023

Outcome should be an updated / enhaced upgrade guide https://doc.owncloud.com/ocis/next/migration/upgrading-ocis.html

Please ping @xoxys for review.

@micbar
Copy link
Contributor

micbar commented Sep 1, 2023

@mmattel FYI

@ScharfViktor
Copy link
Contributor

ScharfViktor commented Sep 1, 2023

Upgrade test process

for upgrade test I use https://github.com/owncloud/ocis/blob/stable-3.0/deployments/examples/ocis_wopi/docker-compose.yml for 3.0.0 version and https://github.com/owncloud/ocis/blob/stable-4.0/deployments/examples/ocis_wopi/docker-compose.yml for 4.0.0 version and of course read docs: https://doc.owncloud.com/ocis/next/migration/upgrading_3.0.0_4.0.0.html

v3.0 and v4.0 comparison

  • MICRO_REGISTRY: "mdns"(v4) - specified in the docs
  • FRONTEND_FULL_TEXT_SEARCH_ENABLED: true(v4) - doesn't specified in the docs
  • REVA_GATEWAY changed ocis:9142 -> com.owncloud.api.gateway - specified in the docs
  • APP_PROVIDER_EXTERNAL_ADDR changed: ocis-appprovider-collabora:9164-> com.owncloud.api.app-provider-collabora - doesn't specified in the docs
  • APP_PROVIDER_SERVICE_NAME: added app-provider-collabora/onlyoffice - doesn't specified in the docs
  • upgrated
    • [] wopiserver image: 9.4.0 -> 10.1.0
    • [] onlyoffice image: onlyoffice/documentserver:7.3.0 -> onlyoffice/documentserver:7.3.3
    • [] collabora image: collabora/code:22.05.10.1.1 -> collabora/code:22.05.14.3.1

remarks:

file couldn't open in collabora using cs3org/wopiserver:v10.0.1 version. v9.4.0 works correct

log:

ERR  WOPI::GetFile [https://wopiserver.owncloud.test/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk/contents?access_token=&access_token_ttl=1693655965000&permission=edit] failed with Status Code: 500| wsd/Storage.cpp:1172

wsd-00001-00128 2023-09-01 11:59:30.630312 +0000 [ docbroker_00f ] ERR  Cannot download document from WOPI storage uri [https://wopiserver.owncloud.test/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk/contents?access_token=&access_token_ttl=1693655965000&permission=edit]. Error: WOPI::GetFile [https://wopiserver.owncloud.test/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk/contents?access_token=&access_token_ttl=1693655965000&permission=edit] failed: {"message": "Failed to fetch file from storage"}| wsd/Storage.cpp:1103

wsd-00001-00128 2023-09-01 11:59:30.630381 +0000 [ docbroker_00f ] ERR  loading document exception: WOPI::GetFile [https://wopiserver.owncloud.test/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk/contents?access_token=&access_token_ttl=1693655965000&permission=edit] failed: {"message": "Failed to fetch file from storage"}| wsd/DocumentBroker.cpp:2510

wsd-00001-00128 2023-09-01 11:59:30.630403 +0000 [ docbroker_00f ] ERR  Failed to add session to [https://wopiserver.owncloud.test:443/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk] with URI [https://wopiserver.owncloud.test/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk?access_token=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VyaWQiOiJleUpoYkdjaU9pSklVekkxTmlJc0luUjVjQ0k2SWtwWFZDSjkuZXlKaGRXUWlPaUp5WlhaaElpd2laWGh3SWpveE5qa3pOalUxT1RZMUxDSnBZWFFpT2pFMk9UTTFOamsxTmpVc0ltbHpjeUk2SW1oMGRIQnpPaTh2YjJOcGN5NXZkMjVqYkc5MVpDNTBaWE4wSWl3aWRYTmxjaUk2ZXlKcFpDSTZleUpwWkhBaU9pSm9kSFJ3Y3pvdkwyOWphWE11YjNkdVkyeHZkV1F1ZEdWemRDSXNJbTl3WVhGMVpWOXBaQ0k2SWpZMU1tUXdaVFl6TFRjeVptRXRORFl4TVMxaE5EazNMVGxsWkRWbVlqWmhNek01WlNJc0luUjVjR1VpT2pGOUxDSjFjMlZ5Ym1GdFpTSTZJbUZrYldsdUlpd2liV0ZwYkNJNkltRmtiV2x1UUdWNFlXMXdiR1V1YjNKbklpd2laR2x6Y0d4aGVWOXVZVzFsSWpvaVFXUnRhVzRpTENKMWFXUmZiblZ0WW1WeUlqbzVPU3dpWjJsa1gyNTFiV0psY2lJNk9UbDlMQ0p6WTI5d1pTSTZleUoxYzJWeUlqcDdJbkpsYzI5MWNtTmxJanA3SW1SbFkyOWtaWElpT2lKcWMyOXVJaXdpZG1Gc2RXVWlPaUpsZVVwM1dWaFNiMGxxYjJsTWVVbzVJbjBzSW5KdmJHVWlPakY5ZlgwLlBPdnpKbklrV01CNWl0bFk1T05qRFpBYzFRTUdKdTRxbjhLNnktZ0ROTVUiLCJ3b3BpdXNlciI6IjY1MmQwZTYzLTcyZmEtNDYxMS1hNDk3LTllZDVmYjZhMzM5ZSE2NTJkMGU2My03MmZhLTQ2MTEtYTQ5Ny05ZWQ1ZmI2YTMzOWVAaHR0cHM6Ly9vY2lzLm93bmNsb3VkLnRlc3QiLCJ1c2VydHlwZSI6InJlZ3VsYXIiLCJmaWxlbmFtZSI6IjY1MmQwZTYzLTcyZmEtNDYxMS1hNDk3LTllZDVmYjZhMzM5ZS9OZXVlIERhdGVpLm9kcyIsImZpbGVpZCI6ImU3MDZiN2MwLWQyNDktNGU0Yi1hYTE3LTRkNmY1ZTAzY2FiZCIsInVzZXJuYW1lIjoiQWRtaW4iLCJ2aWV3bW9kZSI6IlZJRVdfTU9ERV9SRUFEX1dSSVRFIiwiZm9sZGVydXJsIjoiaHR0cHM6Ly9vY2lzLm93bmNsb3VkLnRlc3QvZi8zYmI4MjRlMy1jMjBjLTRmZWQtODdkOS1mZWNhMmMyMjg4OGYkNjUyZDBlNjMtNzJmYS00NjExLWE0OTctOWVkNWZiNmEzMzllIWU3MDZiN2MwLWQyNDktNGU0Yi1hYTE3LTRkNmY1ZTAzY2FiZCIsImVuZHBvaW50IjoiM2JiODI0ZTMtYzIwYy00ZmVkLTg3ZDktZmVjYTJjMjI4ODhmJDY1MmQwZTYzLTcyZmEtNDYxMS1hNDk3LTllZDVmYjZhMzM5ZSIsImFwcG5hbWUiOiJDb2xsYWJvcmEiLCJhcHBlZGl0dXJsIjoiaHR0cHM6Ly9jb2xsYWJvcmEub3duY2xvdWQudGVzdC9icm93c2VyL2MzNjAwMTE4MTUvY29vbC5odG1sPyIsImFwcHZpZXd1cmwiOiJodHRwczovL2NvbGxhYm9yYS5vd25jbG91ZC50ZXN0L2Jyb3dzZXIvYzM2MDAxMTgxNS9jb29sLmh0bWw_IiwiZXhwIjoxNjkzNjU1OTY1LCJpc3MiOiJjczNvcmc6d29waXNlcnZlcjoifQ.9kncuBXc981T0LGjEVn042r48Y1dhfe3aM1UMbjgcr8&access_token_ttl=1693655965000&permission=edit]: WOPI::GetFile [https://wopiserver.owncloud.test/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk/contents?access_token=&access_token_ttl=1693655965000&permission=edit] failed: {"message": "Failed to fetch file from storage"}| wsd/DocumentBroker.cpp:2472

wsd-00001-00128 2023-09-01 11:59:30.630486 +0000 [ docbroker_00f ] ERR  Storage error while starting session on https://wopiserver.owncloud.test:443/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk for socket #28. Terminating connection. Error: WOPI::GetFile [https://wopiserver.owncloud.test/wopi/files/3bb824e3-c20c-4fed-87d9-feca2c22888f%21ZTcwNmI3YzAtZDI0OS00ZTRiLWFhMTctNGQ2ZjVlMDNjYWJk/contents?access_token=&access_token_ttl=1693655965000&permission=edit] failed: {"message": "Failed to fetch file from storage"}| wsd/COOLWSD.cpp:4929

wsd-00001-00128 2023-09-01 11:59:30.631016 +0000 [ docbroker_00f ] ERR  #28: Socket write returned -1 (EPIPE: Broken pipe)| net/Socket.hpp:1442

wsd-00001-00128 2023-09-01 11:59:30.644172 +0000 [ docbroker_00f ] ERR  #29: Read failed, have 0 buffered bytes (ECONNRESET: Connection reset by peer)| net/Socket.hpp:1136

wsd-00001-00128 2023-09-01 11:59:30.644265 +0000 [ docbroker_00f ] WRN  #29: Unassociated Kit (127) disconnected unexpectedly| wsd/COOLWSD.cpp:3409

@xoxys
Copy link
Contributor Author

xoxys commented Sep 1, 2023

Where is MICRO_REGISTRY specified in the docs? Its not in https://doc.owncloud.com/ocis/next/migration/upgrading_3.0.0_4.0.0.html

@ScharfViktor
Copy link
Contributor

yes, on this page we haven't.
indicated and added here https://doc.owncloud.com/ocis/next/deployment/services/env-vars-special-scope.html

@xoxys
Copy link
Contributor Author

xoxys commented Sep 1, 2023

Ok, but this page doesn't even mention what this env var is used for. And things like Only change on supervision of ownCloud Support. in the docs are just bad... No chance that a user knows this is required for office integrations now and according to the docs MICRO_REGISTRY doesn't have a default, so user have to explicitly set it.

@zjean
Copy link

zjean commented Sep 13, 2023

any updates on this? I want to migrate from 3.0 to 4.0.1, but am hesitant to do so.

rhafer added a commit to rhafer/ocis that referenced this issue Sep 19, 2023
For some reason LDAP_BIND_PASSWORD was forgotten when all the other global LDAP_
variables got renamed.

Also marks LDAP_BIND_PASSWORD and LDAP_USER_SCHEMA_ID_IS_OCTETSTRING for removal with 5.0.0

Partial: owncloud#7176
rhafer added a commit that referenced this issue Sep 20, 2023
For some reason LDAP_BIND_PASSWORD was forgotten when all the other global LDAP_
variables got renamed.

Also marks LDAP_BIND_PASSWORD and LDAP_USER_SCHEMA_ID_IS_OCTETSTRING for removal with 5.0.0

Partial: #7176
rhafer added a commit to rhafer/ocis that referenced this issue Sep 21, 2023
For some reason LDAP_BIND_PASSWORD was forgotten when all the other global LDAP_
variables got renamed.

Also marks LDAP_BIND_PASSWORD and LDAP_USER_SCHEMA_ID_IS_OCTETSTRING for removal with 5.0.0

Partial: owncloud#7176
(cherry picked from commit 8f284e7)
rhafer added a commit to rhafer/ocis that referenced this issue Sep 21, 2023
LDAP_BIND_PASSWORD was accidently removed in previous commit

Partial: owncloud#7176
rhafer added a commit that referenced this issue Sep 21, 2023
LDAP_BIND_PASSWORD was accidently removed in previous commit

Partial: #7176
rhafer added a commit to rhafer/ocis that referenced this issue Sep 21, 2023
LDAP_BIND_PASSWORD was accidently removed in previous commit

Partial: owncloud#7176
(cherry picked from commit 8f262c1)
ownclouders pushed a commit that referenced this issue Sep 21, 2023
LDAP_BIND_PASSWORD was accidently removed in previous commit

Partial: #7176
rhafer added a commit that referenced this issue Sep 21, 2023
For some reason LDAP_BIND_PASSWORD was forgotten when all the other global LDAP_
variables got renamed.

Also marks LDAP_BIND_PASSWORD and LDAP_USER_SCHEMA_ID_IS_OCTETSTRING for removal with 5.0.0

Partial: #7176
(cherry picked from commit 8f284e7)
rhafer added a commit that referenced this issue Sep 21, 2023
LDAP_BIND_PASSWORD was accidently removed in previous commit

Partial: #7176
(cherry picked from commit 8f262c1)
ownclouders pushed a commit that referenced this issue Sep 21, 2023
LDAP_BIND_PASSWORD was accidently removed in previous commit

Partial: #7176
(cherry picked from commit 8f262c1)
@ScharfViktor
Copy link
Contributor

OCIS_LDAP_BIND_PASSWORD not working and must use LDAP_BIND_PASSWORD while all other LDAP env vars were prefixed with OCIS_

fixed here #7313

It will be available again in ocis 4.0.2

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

No branches or pull requests

4 participants