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

Pre-existing ex_app resource doesn't seem to work #26

Closed
marcelklehr opened this issue Dec 13, 2024 · 2 comments · Fixed by #35
Closed

Pre-existing ex_app resource doesn't seem to work #26

marcelklehr opened this issue Dec 13, 2024 · 2 comments · Fixed by #35
Labels
bug Something isn't working

Comments

@marcelklehr
Copy link
Member

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Install exapp
  2. Setup a flow with a script that uses AppAPi auth from the resource
  3. Nextcloud complains that user flow_app cannot be found

Expected behaviour

App API auth should work

Actual behaviour

Auth fails

Server configuration

Web server: Apache/Nginx

Database: MySQL/Maria/SQLite/PostgreSQL

PHP version: 8.1/8.2/8.3

Nextcloud version: 30, master

@marcelklehr marcelklehr added the bug Something isn't working label Dec 13, 2024
@bigcat88
Copy link
Member

Cannot reproduce this.

For test a take script example from here:

https://raw.githubusercontent.com/nextcloud/flow_scripts/refs/heads/main/actions/List_calendars.py

Pressed +Resource button, changed the script to use resource from import:

Image

The request is send fine to the Nextсloud instance.

From what I modified, I manually added nextcloud_url to the resource so that it would point to the NC in my dev setup.

So the question is, should I add the initialization of "nextcloud_url" or "baseUrl" (I don't care about the name, but we have the previous variables in Snake Case) - when initializing Windmill?

Or any other ideas? I remember we originally required admins to manually fill in baseUrl when we released the first version of Windmill, but for some reason I can't find where to edit it in the Windmill UI now, and it's not in the documentation (or I'm blind and tired):

https://docs.nextcloud.com/server/latest/admin_manual/windmill_workflows/index.html

cc @julien-nc @marcelklehr

@marcelklehr
Copy link
Member Author

So the question is, should I add the initialization of "nextcloud_url" or "baseUrl" (I don't care about the name, but we have the previous variables in Snake Case) - when initializing Windmill?

Yeah, that would be good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants