meissa
/
forgejo
7
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
forgejo-federated-star
forgejo
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from 'b24a965b81'
${ noResults }
forgejo/services/webhook
History
qwerty287 9d943bf374
Add missing `X-Total-Count` and fix some related bugs (#17968) 
* Add missing `X-Total-Count` and fix some related bugs

Adds `X-Total-Count` header to APIs that return a list but doesn't have it yet.
Fixed bugs:
* not returned after reporting error (39eb82446c/routers/api/v1/user/star.go (L70))
* crash with index out of bounds, API issue/issueSubscriptions

I also found various endpoints that return lists but do not apply/support pagination yet:
```
/repos/{owner}/{repo}/issues/{index}/labels
/repos/{owner}/{repo}/issues/comments/{id}/reactions
/repos/{owner}/{repo}/branch_protections
/repos/{owner}/{repo}/contents
/repos/{owner}/{repo}/hooks/git
/repos/{owner}/{repo}/issue_templates
/repos/{owner}/{repo}/releases/{id}/assets
/repos/{owner}/{repo}/reviewers
/repos/{owner}/{repo}/teams
/user/emails
/users/{username}/heatmap
```
If this is not expected, an new issue should be opened.

Closes #13043

* fmt

* Update routers/api/v1/repo/issue_subscription.go

Co-authored-by: KN4CK3R <admin@oldschoolhack.me>

* Use FindAndCount

Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
Co-authored-by: 6543 <6543@obermui.de>
 		3 years ago
..
deliver.go Use `hostmatcher` to replace `matchlist`, improve security (#17605) 3 years ago
deliver_test.go A better go code formatter, and now `make fmt` can run in Windows (#17684) 3 years ago
dingtalk.go Multiple Escaping Improvements (#17551) 3 years ago
dingtalk_test.go Move webhook into models/webhook/ (#17579) 3 years ago
discord.go Multiple Escaping Improvements (#17551) 3 years ago
discord_test.go Move webhook into models/webhook/ (#17579) 3 years ago
feishu.go Move webhook into models/webhook/ (#17579) 3 years ago
feishu_test.go Move webhook into models/webhook/ (#17579) 3 years ago
general.go Multiple Escaping Improvements (#17551) 3 years ago
general_test.go Add tests for all webhooks (#16214) 3 years ago
main_test.go Add missing `X-Total-Count` and fix some related bugs (#17968) 3 years ago
matrix.go Multiple Escaping Improvements (#17551) 3 years ago
matrix_test.go Move webhook into models/webhook/ (#17579) 3 years ago
msteams.go Multiple Escaping Improvements (#17551) 3 years ago
msteams_test.go Move webhook into models/webhook/ (#17579) 3 years ago
payloader.go Move webhook into models/webhook/ (#17579) 3 years ago
slack.go Move webhook into models/webhook/ (#17579) 3 years ago
slack_test.go Move webhook into models/webhook/ (#17579) 3 years ago
telegram.go Move webhook into models/webhook/ (#17579) 3 years ago
telegram_test.go Move webhook into models/webhook/ (#17579) 3 years ago
webhook.go Move repository model into models/repo (#17933) 3 years ago
webhook_test.go Move repository model into models/repo (#17933) 3 years ago
wechatwork.go Move webhook into models/webhook/ (#17579) 3 years ago