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

FEAT: OAuth system supports api-key #1168

Merged
merged 34 commits into from
Mar 28, 2024
Merged

Conversation

Ago327
Copy link
Contributor

@Ago327 Ago327 commented Mar 21, 2024

The built-in permission system now supports API key authentication.

Fixes #918

@XprobeBot XprobeBot added this to the v0.9.4 milestone Mar 21, 2024
@ChengjieLi28 ChengjieLi28 changed the title Api key feature FEAT: OAuth system supports api-key Mar 21, 2024
@ChengjieLi28
Copy link
Contributor

ChengjieLi28 commented Mar 21, 2024

What about command-line and curl users?
I think you should add api-key for every interface. And also, make every request compatiable with OpenAI.

@XprobeBot XprobeBot modified the milestones: v0.9.4, v0.9.5 Mar 21, 2024
xinference/deploy/cmdline.py Outdated Show resolved Hide resolved
xinference/client/restful/restful_client.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
xinference/api/oauth2/auth_service.py Outdated Show resolved Hide resolved
@ChengjieLi28 ChengjieLi28 merged commit b44c2ce into xorbitsai:main Mar 28, 2024
8 of 12 checks passed
qinxuye pushed a commit to qinxuye/inference that referenced this pull request Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Any further plan to add access key for rest API call
3 participants