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

[bug] Issue when trying to install external app via appstore #517

Open
R0Wi opened this issue Feb 17, 2025 · 0 comments · May be fixed by #518
Open

[bug] Issue when trying to install external app via appstore #517

R0Wi opened this issue Feb 17, 2025 · 0 comments · May be fixed by #518

Comments

@R0Wi
Copy link
Member

R0Wi commented Feb 17, 2025 •
edited
Loading

Describe the bug

When trying to install an ExApp via Nextcloud Appstore (UI), the app gets stuck and then fails

I added a few debug log statements and found this:

{
   "reqId":"RxuK0gxgmrpPF0sscwUP",
   "level":1,
   "time":"2025-02-16T13:21:07+00:00",
   "remoteAddr":"172.25.0.1",
   "user":"admin",
   "app":"app_api",
   "method":"POST",
   "url":"/apps/app_api/apps/enable/workflow_ocr_backend",
   "message":"Calling occ(directory=null): 'app_api:app:register' '--silent' 'workflow_ocr_backend' '' '' --no-ansi --no-warnings",
   "userAgent":"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/132.0.0.0 Safari/537.36",
   "version":"31.0.0.15",
   "data":{
      "app":"app_api"
   },
   "id":"67b39c0b146aa"
}

{
   "reqId":"RxuK0gxgmrpPF0sscwUP",
   "level":0,
   "time":"2025-02-16T13:21:07+00:00",
   "remoteAddr":"172.25.0.1",
   "user":"admin",
   "app":"app_api",
   "method":"POST",
   "url":"/apps/app_api/apps/enable/workflow_ocr_backend",
   "message":"occ error output: \n                                                                               \n  Too many arguments to \"app_api:app:register\" command, expected arguments \"a  \n  ppid\" \"daemon-config-name\".                                                  \n                                                                               \n\napp_api:app:register [--force-scopes] [--info-xml INFO-XML] [--json-info JSON-INFO] [--wait-finish] [--silent] [--test-deploy-mode] [--env [ENV]] [--mount [MOUNT]] [--] <appid> [<daemon-config-name>]\n\n",
   "userAgent":"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/132.0.0.0 Safari/537.36",
   "version":"31.0.0.15",
   "data":{
      "app":"app_api"
   },
   "id":"67b39c0b146a1"
}

To me it seems that the occ command, which is called under the hood, fails, because the parameters are not being setup correctly (note the "empty" parameters)

Steps/Code to Reproduce

  1. Install NC31
  2. Install AppApi and Deploy daemon
  3. Try to install (any) ExApp from the App Store

Expected Results

App should be installed

Actual Results

See above

Setup configuration

NC31
AppApi 5.0.2
R0Wi added a commit that referenced this issue Feb 17, 2025
@R0Wi
Fix parameter preparation for occ command
275486f 
* Fix for #517

Signed-off-by: Robin Windey <ro.windey@gmail.com>
@R0Wi R0Wi linked a pull request Feb 17, 2025 that will close this issue
Open
@R0Wi R0Wi linked a pull request Feb 17, 2025 that will close this issue
Fix parameter preparation for occ command #518
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix parameter preparation for occ command nextcloud/app_api
1 participant
@R0Wi