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

Word WOPI double present in env var in app-provider #3709

Closed
mmattel opened this issue May 6, 2022 · 2 comments
Closed

Word WOPI double present in env var in app-provider #3709

mmattel opened this issue May 6, 2022 · 2 comments

Comments

@mmattel
Copy link
Contributor

mmattel commented May 6, 2022

In extension app-provider
Why do we need to have the word WOPI double present in the env name?

APP_PROVIDER_WOPI_WOPI_SERVER_IOP_SECRET
APP_PROVIDER_WOPI_WOPI_SERVER_EXTERNAL_URL

@butonic @micbar @wkloucek

@butonic
Copy link
Member

butonic commented May 6, 2022

The duplication of WOPI is caused by the sequence of driver names and their config option names. We could introduce two __ as a separator to indicate these boundaries.

@micbar micbar mentioned this issue May 10, 2022
45 tasks
@micbar
Copy link
Contributor

micbar commented Jul 4, 2022

Close as won't fix.

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

No branches or pull requests

3 participants