nic-chen
Follow
Create your own GitHub profile
Sign up for your own profile on GitHub, the best place to host code, manage projects, and build software alongside 50 million developers.
Sign upPinned
551 contributions in the last year
Contribution activity
December 2020
Created 16 commits in 1 repository
Created a pull request in apache/apisix-dashboard that received 6 comments
feat: use Swagger to maintain API document for Manager API
Please answer these questions before submitting a pull request Why submit this pull request? Bugfix New feature provided Improve perform…
+691
−2
•
6
comments
Opened 15 other pull requests in 1 repository
apache/apisix-dashboard
2
open
11
merged
2
closed
- fix: remove the dependency on git from the build script
- ci: display more meaningful information when running E2E test cases failed
- fix: PATCH method bug
- fix: fix router name conflict in route_service_upstream_test.go
- feat: add access log for Manager API
- fix: Use JSONSchema to verify the original data submitted by users
- chore: enrich error log
- ci: increase the sleep time to fix e2e test unstable issue
- fix: hack to fix the checking of `remote_addrs`
- fix: sync json schema to fix the checking of `remote_addrs`
- ci: run the E2E test cases with APISIX 2.1
- fix: upstream node can't display when the node's weight is set to 0
- chore: change the default port of manager api from 8080 to 9000
- CD: auto deploy codes on the master branch to the online demo
- ci: should deploy code in master branch to the online demo
Reviewed 20 pull requests in 4 repositories
apache/apisix-dashboard 17 pull requests
- feat: add a unit test for consumer and remove implicit init
- feat: remove husky from the web
- revert: "fix: delete POST method in /apisix/admin/consumer"
- release: new version 2.1.1
- feat: change proxy target to localhost when test
- fix: spelling
- test: e2e test chash upstream with key (wrong key, vars)
- test: e2e test balancer (roundrobin upstream)
- feat: aligned comments conf.yaml
- feat: add etcd basic auth support
- feat: add version info into manager-api
- test: e2e test chash upstream hash on (consumer, header, cookie)
- test: fix the problem that consumer test cases cause CI failure (E2E)
- test: e2e test chash upstream with key (query_string, arg_xxx)
- test: config route with management fileds (E2E)
- test: e2e test chash upstream with key (remote_addr)
- fix: snowflake remove network dependency
api7/lua-resty-etcd 1 pull request
apache/apisix-ingress-controller 1 pull request
apache/apisix 1 pull request
Created an issue in apache/apisix-dashboard that received 9 comments
bug: when user not fill in `remote address` , `remote_addrs` should be [], but not [""]
Bug report
How to Reproduce
use dashboard to create a route without filling in remote address
wait a few minutes, will find some error log like
9
comments
Opened 8 other issues in 3 repositories
apache/apisix-dashboard
5
open
1
closed
- style: code optimization for `api/filters/schema.go`
- feat: support generate and convert format api doc in Makefile
- feat: collect code coverage for backend e2e test
- feat: need a tool to check version before release
- bug: JSON schema verification should be performed on the original data submitted by the user
- test: need to do custom check for plugins