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

参数交换位置不能使用 #18

Open
clansty opened this issue Oct 27, 2023 · 2 comments
Open

参数交换位置不能使用 #18

clansty opened this issue Oct 27, 2023 · 2 comments
Assignees
Labels
feature New feature or request pull request welcome Extra attention is needed

Comments

@clansty
Copy link

clansty commented Oct 27, 2023

https://b23.wtf/api?full=[url]&status=200 如果换成 https://b23.wtf/api?status=200&full=[url] 会 400 bad request

@nicholascw
Copy link
Owner

nicholascw commented Nov 3, 2023

It's expected as this is the only API and the api?full= string is hard coded in the source code as shown in server.c, Would need a major overhaul to "properly implement" the api argument parsing implementation to fix this.

@nicholascw nicholascw added feature New feature or request pull request welcome Extra attention is needed bug Something isn't working and removed feature New feature or request pull request welcome Extra attention is needed labels Nov 3, 2023
@nicholascw nicholascw reopened this Nov 3, 2023
@nicholascw nicholascw added feature New feature or request pull request welcome Extra attention is needed and removed bug Something isn't working labels Nov 3, 2023
@nicholascw
Copy link
Owner

the original issue should have been resolved by 71ace7f but the issue is kept open as it is a workaround and the refactor should be done some time in the future.

@nicholascw nicholascw self-assigned this Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request pull request welcome Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants