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

[Bug]: missing nginx path for SDK handlers #2899

Closed
1 task done
kaleocheng opened this issue Oct 18, 2024 · 0 comments
Closed
1 task done

[Bug]: missing nginx path for SDK handlers #2899

kaleocheng opened this issue Oct 18, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@kaleocheng
Copy link
Contributor

kaleocheng commented Oct 18, 2024

Is there an existing issue for the same bug?

  • I have checked the existing issues.

Branch name

main

Commit ID

526fcbb

Other environment information

No response

Actual behavior

can not call the sdk api with path {domain}/api/v1/{sdk handler}

Expected behavior

No response

Steps to reproduce

curl -X POST "http://{nginx-ip}/api/v1/dify/retrieval" -H "Authorization: Bearer ragflow-token" -H "Content-Type: application/json"

got not found

Additional information

need to update docker/nginx/ragflow.conf

-    location /v1 {
+    location ~ ^/(v1|api) {
         proxy_pass http://ragflow:9380;
         include proxy.conf;
      }

but I'm not sure if it's intentional to exclude /api here.

@kaleocheng kaleocheng added the bug Something isn't working label Oct 18, 2024
kaleocheng added a commit to kaleocheng/ragflow that referenced this issue Oct 18, 2024
KevinHuSh pushed a commit that referenced this issue Oct 21, 2024
### What problem does this PR solve?

add the nginx path `/api` for sdk handlers 

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
Halfknow pushed a commit to Halfknow/ragflow that referenced this issue Nov 11, 2024
### What problem does this PR solve?

add the nginx path `/api` for sdk handlers 

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
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

No branches or pull requests

2 participants