juzhiyuan opened a new pull request #140:
URL: https://github.com/apache/apisix-website/pull/140
Changes:
added deploy CD
This is an automated message from the Apache Git Service.
To respond to the message, please log on
hnlq715 opened a new issue #3303:
URL: https://github.com/apache/apisix/issues/3303
As title
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a change to branch juzhiyuan-patch-1
in repository https://gitbox.apache.org/repos/asf/apisix-website.git.
at 177879a Update deploy.yml
This branch includes the following new commits:
new 177879a
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a commit to branch juzhiyuan-patch-1
in repository https://gitbox.apache.org/repos/asf/apisix-website.git
commit 177879ad2b543984820be379278b2af087fd68b9
Author: 琚致远
AuthorDate: Fri Jan 15 15:59:05 2021 +0800
Firstsawyou commented on issue #3301:
URL: https://github.com/apache/apisix/issues/3301#issuecomment-760718034
It may be that the file size of the proxy is limited.
@spacewander Please help to look at this problem when you have time.
idbeta commented on issue #3302:
URL: https://github.com/apache/apisix/issues/3302#issuecomment-760716918
Welcome to pr @admin-xl-2048
This is an automated message from the Apache Git Service.
To respond to the message,
admin-xl-2048 commented on issue #3302:
URL: https://github.com/apache/apisix/issues/3302#issuecomment-760715462
There is a small error under this file: CD apache-apisix-2.2 should be
changed to CD apisix-2.2
This is an
admin-xl-2048 opened a new issue #3302:
URL: https://github.com/apache/apisix/issues/3302
# Improve Docs
## Please describe which part of docs should be improved or typo fixed
A clear and concise description of what you want and what your use case is.
## Describe the
admin-xl-2048 commented on issue #1305:
URL:
https://github.com/apache/apisix-dashboard/issues/1305#issuecomment-760711148
非常感谢，按照这个方法我已经解决掉了。
--原始邮件--
发件人:
hnlq715 commented on a change in pull request #3300:
URL: https://github.com/apache/apisix/pull/3300#discussion_r557947934
##
File path: apisix/balancer/ewma.lua
##
@@ -25,80 +28,102 @@ local lrucache_trans_format = core.lrucache.new({
ttl = 300, count = 256
})
-
hnlq715 commented on a change in pull request #3300:
URL: https://github.com/apache/apisix/pull/3300#discussion_r557947468
##
File path: apisix/balancer/ewma.lua
##
@@ -25,80 +28,102 @@ local lrucache_trans_format = core.lrucache.new({
ttl = 300, count = 256
})
-
guoqqqi opened a new pull request #1306:
URL: https://github.com/apache/apisix-dashboard/pull/1306
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [x] Bugfix
- [ ] New feature provided
- [ ] Improve performance
- [ ]
membphis commented on a change in pull request #3300:
URL: https://github.com/apache/apisix/pull/3300#discussion_r557934625
##
File path: apisix/balancer/ewma.lua
##
@@ -25,80 +28,102 @@ local lrucache_trans_format = core.lrucache.new({
ttl = 300, count = 256
})
-
spacewander commented on issue #3281:
URL: https://github.com/apache/apisix/issues/3281#issuecomment-760703943
By saying `switch to LuaJIT only`, I mean two things:
1. install Luarocks from source, and avoid installing Lua
2. change the `#!` in `bin/apisix` to use luajit.
pengtaochang opened a new issue #3301:
URL: https://github.com/apache/apisix/issues/3301
### Issue description
I user apisix proxy a request which using for upload big file (5MB around)
, the request raise Error：socket hang up.
I had test my api server, upload and download api
membphis commented on pull request #3300:
URL: https://github.com/apache/apisix/pull/3300#issuecomment-760701953
nice PR @hnlq715
This is an automated message from the Apache Git Service.
To respond to the message, please
juzhiyuan commented on issue #1305:
URL:
https://github.com/apache/apisix-dashboard/issues/1305#issuecomment-760701021
Hi, this issue is fixed in the master branch. You could simply do this
patch[1] for your codes.
[1]
juzhiyuan closed issue #1305:
URL: https://github.com/apache/apisix-dashboard/issues/1305
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
hnlq715 opened a new pull request #3300:
URL: https://github.com/apache/apisix/pull/3300
### What this PR does / why we need it:
Details can be found in #3211
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new features
membphis commented on issue #3281:
URL: https://github.com/apache/apisix/issues/3281#issuecomment-760697698
@fukiki
what is your OS name and version? what is the Luarocks version? what is the
Lua version?
we need those information.
membphis commented on a change in pull request #3282:
URL: https://github.com/apache/apisix/pull/3282#discussion_r557922669
##
File path: doc/how-to-build.md
##
@@ -109,6 +109,49 @@ Makefile rules:
Environment variable can be used to configure APISIX. Please take a look at
starsz commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557915888
##
File path: api/internal/filter/authentication.go
##
@@ -17,62 +17,79 @@
package filter
import (
+ "errors"
"net/http"
starsz commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557900139
##
File path: api/internal/filter/authentication.go
##
@@ -17,62 +17,79 @@
package filter
import (
+ "errors"
"net/http"
spacewander commented on issue #3281:
URL: https://github.com/apache/apisix/issues/3281#issuecomment-760691640
@fukiki @liuxiran
The documentation need to be updated. I think we should mention that only
LuaJIT is supported. Unless you know Lua well, it is not obvious to find the
nic-chen merged pull request #1301:
URL: https://github.com/apache/apisix-dashboard/pull/1301
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
This is an automated email from the ASF dual-hosted git repository.
chenjunxu pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git.
from b9a0227 fix: Support string type for the script field in Route (#1289)
add 84af5f1 chore: sync
nic-chen closed issue #1227:
URL: https://github.com/apache/apisix-dashboard/issues/1227
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
spacewander merged pull request #3287:
URL: https://github.com/apache/apisix/pull/3287
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
This is an automated email from the ASF dual-hosted git repository.
spacewander pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git
The following commit(s) were added to refs/heads/master by this push:
new bbbdf58 refactor: the parent of upstream
codecov-io commented on pull request #1304:
URL: https://github.com/apache/apisix-dashboard/pull/1304#issuecomment-760687643
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1304?src=pr=h1)
Report
> Merging
idbeta commented on issue #1288:
URL:
https://github.com/apache/apisix-dashboard/issues/1288#issuecomment-760686696
> @idbeta What about preparing some stuffs like debuginfo.rpm when compiling
it.
sorry, I don't quite understand what you mean.
I don't think `reduce` is necessary
admin-xl-2048 opened a new issue #1305:
URL: https://github.com/apache/apisix-dashboard/issues/1305
# Feature request
## Please describe your feature
A clear and concise description of what you want and what your use case is.
## Describe the solution you'd like
A
starsz commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557900139
##
File path: api/internal/filter/authentication.go
##
@@ -17,62 +17,79 @@
package filter
import (
+ "errors"
"net/http"
idbeta opened a new pull request #1304:
URL: https://github.com/apache/apisix-dashboard/pull/1304
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [x] E2E
- [ ] New feature provided
- [ ] Improve performance
- [ ] Backport
liuxiran commented on issue #1303:
URL:
https://github.com/apache/apisix-dashboard/issues/1303#issuecomment-760680814
> @liuxiran Thanks for your advice. I think we should add a doc after those
issues were solved:
>
> 1. remove the etcd dependency in the unit test #1161
> 2. use
tokers commented on a change in pull request #1102:
URL: https://github.com/apache/apisix-dashboard/pull/1102#discussion_r557885773
##
File path: api/internal/handler/data_loader/import.go
##
@@ -0,0 +1,468 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
liuxiran edited a comment on issue #3281:
URL: https://github.com/apache/apisix/issues/3281#issuecomment-760675043
Once I came to the same issue, and thanks to @spacewander, he told me that I
have to use `Lua5.1` instead, and then I tried to find Lua5.1 for
fedora32(which is the os of my
liuxiran commented on issue #3281:
URL: https://github.com/apache/apisix/issues/3281#issuecomment-760675043
Once I came to the same issue, and thanks to @spacewander, he told me that I
have to use `Lua5.1` instead, and then I tried to find Lua5.1 for fedora32,
unfortunately there was not
tokers commented on issue #1288:
URL:
https://github.com/apache/apisix-dashboard/issues/1288#issuecomment-760663667
> > > I'm not sure whether this is an good idea since for now i don't think
the manager api is stable enough, once it crashes, debug info is important to
dump the stack,
starsz commented on issue #1303:
URL:
https://github.com/apache/apisix-dashboard/issues/1303#issuecomment-760662897
@liuxiran Thanks for your advice. I think we should add a doc after those
issues were solved:
1. remove the etcd dependency in the unit test #1161
2. use ginko
tokers commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557875463
##
File path: api/internal/filter/authentication.go
##
@@ -17,62 +17,79 @@
package filter
import (
+ "errors"
"net/http"
starsz closed issue #1281:
URL: https://github.com/apache/apisix-dashboard/issues/1281
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
tokers commented on issue #3211:
URL: https://github.com/apache/apisix/issues/3211#issuecomment-760660477
> -- | --
OK, My Nginx version is old :)
This is an automated message from the Apache Git Service.
To respond
codecov-io edited a comment on pull request #1302:
URL: https://github.com/apache/apisix-dashboard/pull/1302#issuecomment-760639257
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1302?src=pr=h1)
Report
> Merging
liuxiran commented on a change in pull request #1298:
URL: https://github.com/apache/apisix-dashboard/pull/1298#discussion_r557873105
##
File path: web/src/global.less
##
@@ -21,6 +21,7 @@ body,
#root {
height: 100%;
word-break: break-word;
+ background-color: #f0f2f5;
nic-chen removed a comment on pull request #1301:
URL: https://github.com/apache/apisix-dashboard/pull/1301#issuecomment-760588574
@juzhiyuan @LiteSun
ci failed:
https://github.com/apache/apisix-dashboard/pull/1301/checks?check_run_id=1705633221
It seems that the schema cannot
codecov-io edited a comment on pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#issuecomment-760587036
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1296?src=pr=h1)
Report
> Merging
fukiki commented on issue #3281:
URL: https://github.com/apache/apisix/issues/3281#issuecomment-760646065
@spacewander
I understand the reason of the mistake and the solution, thanks for your
explain.
I still wish that the document could add relevant content, to avoid
subsequent
liuxiran edited a comment on issue #1278:
URL:
https://github.com/apache/apisix-dashboard/issues/1278#issuecomment-760083931
> Once we auto fill upstream info from service, a new upstream object would
be bound to that route unexpectedly IMO
you mean we should omit the `upstream` in
moonming commented on pull request #177:
URL:
https://github.com/apache/apisix-ingress-controller/pull/177#issuecomment-760643686
great， we can post to apisix.apache.org too
This is an automated message from the Apache Git
juzhiyuan merged pull request #1289:
URL: https://github.com/apache/apisix-dashboard/pull/1289
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
juzhiyuan closed issue #1275:
URL: https://github.com/apache/apisix-dashboard/issues/1275
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git
The following commit(s) were added to refs/heads/master by this push:
new b9a0227 fix: Support string type
juzhiyuan commented on issue #1303:
URL:
https://github.com/apache/apisix-dashboard/issues/1303#issuecomment-760642013
cc @nic-chen @starsz
This is an automated message from the Apache Git Service.
To respond to the
imjoey commented on pull request #1289:
URL: https://github.com/apache/apisix-dashboard/pull/1289#issuecomment-760641858
> But e2e test failed.
@nic-chen @gxthrj CI passed, PTAL. Thanks.
This is an automated message
codecov-io edited a comment on pull request #1301:
URL: https://github.com/apache/apisix-dashboard/pull/1301#issuecomment-760282606
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1301?src=pr=h1)
Report
> Merging
juzhiyuan commented on issue #1300:
URL:
https://github.com/apache/apisix-dashboard/issues/1300#issuecomment-760640883
failed with English.
```sh
curl 'http://139.217.190.60/apisix/admin/routes' \
-H 'Content-Type: application/json;charset=UTF-8' \
--data-binary
codecov-io commented on pull request #1302:
URL: https://github.com/apache/apisix-dashboard/pull/1302#issuecomment-760639257
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1302?src=pr=h1)
Report
> Merging
starsz commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557853660
##
File path: api/cmd/managerapi.go
##
@@ -19,6 +19,7 @@ package cmd
import (
"context"
"fmt"
+
tokers commented on pull request #177:
URL:
https://github.com/apache/apisix-ingress-controller/pull/177#issuecomment-760635408
@membphis @moonming @gxthrj Please take a look when you have time.
This is an automated message
tokers opened a new pull request #177:
URL: https://github.com/apache/apisix-ingress-controller/pull/177
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [ ] Bugfix
- [ ] New feature provided
- [ ] Improve performance
- [ ]
starsz commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557852288
##
File path: api/go.mod
##
@@ -21,6 +21,7 @@ require (
github.com/gin-contrib/sessions v0.0.3
github.com/gin-contrib/static
starsz commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557852183
##
File path: api/cmd/managerapi.go
##
@@ -62,7 +63,7 @@ func NewManagerAPICommand() *cobra.Command {
var newMws
nic-chen commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557847807
##
File path: api/cmd/managerapi.go
##
@@ -62,7 +63,7 @@ func NewManagerAPICommand() *cobra.Command {
var
nic-chen commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557846111
##
File path: api/go.mod
##
@@ -21,6 +21,7 @@ require (
github.com/gin-contrib/sessions v0.0.3
github.com/gin-contrib/static
nic-chen commented on a change in pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#discussion_r557845884
##
File path: api/cmd/managerapi.go
##
@@ -19,6 +19,7 @@ package cmd
import (
"context"
"fmt"
+
liuxiran opened a new issue #1303:
URL: https://github.com/apache/apisix-dashboard/issues/1303
# Feature request
## Please describe your feature
Recently when my partners and I tried to finish the the issues of backend,
we always found troubles in writting unit test cases and
nic-chen merged pull request #3299:
URL: https://github.com/apache/apisix/pull/3299
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
This is an automated email from the ASF dual-hosted git repository.
chenjunxu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git
The following commit(s) were added to refs/heads/master by this push:
new 5bdfb8f chore: remove injected field of
Jaycean commented on pull request #1102:
URL: https://github.com/apache/apisix-dashboard/pull/1102#issuecomment-760622249
> > > > before we import new route data, do we need to clean the old route
first?
> > >
> > >
> > > I think that the original data should not be modified,
spacewander closed issue #1749:
URL: https://github.com/apache/apisix/issues/1749
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
spacewander commented on issue #1749:
URL: https://github.com/apache/apisix/issues/1749#issuecomment-760620879
Consider solved
This is an automated message from the Apache Git Service.
To respond to the message, please log
spacewander closed issue #3261:
URL: https://github.com/apache/apisix/issues/3261
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
spacewander opened a new pull request #3298:
URL: https://github.com/apache/apisix/pull/3298
Signed-off-by: spacewander
### What this PR does / why we need it:
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new
idbeta commented on issue #1288:
URL:
https://github.com/apache/apisix-dashboard/issues/1288#issuecomment-760605744
> > I'm not sure whether this is an good idea since for now i don't think
the manager api is stable enough, once it crashes, debug info is important to
dump the stack,
membphis commented on issue #1300:
URL:
https://github.com/apache/apisix-dashboard/issues/1300#issuecomment-760613921
> Chinese as the key is a normal requirement IMHO, what's your opinion?
> * If Chinese is supported, it should be better to also support all other
non English languages
juzhiyuan commented on issue #1300:
URL:
https://github.com/apache/apisix-dashboard/issues/1300#issuecomment-760612472
thanks for your point! @gxthrj please take a look at this issue, too.
This is an automated message from
LiteSun commented on pull request #1302:
URL: https://github.com/apache/apisix-dashboard/pull/1302#issuecomment-760603783
> just to make sure that this pr could also solves #1278 , if so I will move
to other issues, thanks ^-^
>
> cc @LiteSun @juzhiyuan
According to the
codecov-io commented on pull request #1054:
URL: https://github.com/apache/apisix-dashboard/pull/1054#issuecomment-760612299
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1054?src=pr=h1)
Report
> Merging
membphis merged pull request #3278:
URL: https://github.com/apache/apisix/pull/3278
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
This is an automated email from the ASF dual-hosted git repository.
membphis pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git.
from 45f7dd6 feat: benchmark support specify upstream count (#3292)
add d1d6533 feat: support hash on
nic-chen opened a new pull request #3299:
URL: https://github.com/apache/apisix/pull/3299
### What this PR does / why we need it:
Injected `$comment` into properties will cause the schema to fail to load,
so need to move it.
### Pre-submission checklist:
* [x] Did
membphis merged pull request #3292:
URL: https://github.com/apache/apisix/pull/3292
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
This is an automated email from the ASF dual-hosted git repository.
membphis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git
The following commit(s) were added to refs/heads/master by this push:
new 45f7dd6 feat: benchmark support specify
imjoey commented on issue #1300:
URL:
https://github.com/apache/apisix-dashboard/issues/1300#issuecomment-760595948
Yep, I slightly changed my mind and we should confirm the following two
things fine, then we can add support.
- If Chinese is supported, it should be better to also
spacewander commented on issue #3225:
URL: https://github.com/apache/apisix/issues/3225#issuecomment-760591745
The `lua-resty-etcd` already supports disabling tls verify:
https://github.com/apache/apisix/blob/5482a4bf72b33eabb83fd4bf396a222366c589f3/conf/config-default.yaml#L192-L193
spacewander closed issue #3225:
URL: https://github.com/apache/apisix/issues/3225
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
hnlq715 edited a comment on issue #3211:
URL: https://github.com/apache/apisix/issues/3211#issuecomment-760586816
@tokers
Nginx features are merged into open source version smoothly :-)
http://nginx.org/en/docs/http/ngx_http_upstream_module.html#zone
```
Syntax: | zone name
starsz commented on issue #1288:
URL:
https://github.com/apache/apisix-dashboard/issues/1288#issuecomment-760588681
> I'm not sure whether this is an good idea since for now i don't think the
manager api is stable enough, once it crashes, debug info is important to dump
the stack,
nic-chen commented on pull request #1301:
URL: https://github.com/apache/apisix-dashboard/pull/1301#issuecomment-760588574
@juzhiyuan @LiteSun
ci failed:
https://github.com/apache/apisix-dashboard/pull/1301/checks?check_run_id=1705633221
It seems that the schema cannot be
spacewander commented on issue #3283:
URL: https://github.com/apache/apisix/issues/3283#issuecomment-760587745
Sorry, it is my mistake...
This is an automated message from the Apache Git Service.
To respond to the message,
pengtaochang commented on issue #3294:
URL: https://github.com/apache/apisix/issues/3294#issuecomment-760587010
Thank you very much, you solved my problem perfectly.
This is an automated message from the Apache Git Service.
codecov-io commented on pull request #1296:
URL: https://github.com/apache/apisix-dashboard/pull/1296#issuecomment-760587036
#
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1296?src=pr=h1)
Report
> Merging
hnlq715 commented on issue #3211:
URL: https://github.com/apache/apisix/issues/3211#issuecomment-760586816
@tokers
Nginx features are merged into open source version smoothly :-)
http://nginx.org/en/docs/http/ngx_http_upstream_module.html#zone
```
Syntax: | state file;
starsz commented on issue #1300:
URL:
https://github.com/apache/apisix-dashboard/issues/1300#issuecomment-760582566
I think we should support Chinese labels. This will be more convenient for
some companies.
In another way, we only need to update the validate schema, then we can
Jaycean commented on a change in pull request #1245:
URL: https://github.com/apache/apisix-dashboard/pull/1245#discussion_r557803573
##
File path: api/internal/handler/route_export/route_export.go
##
@@ -0,0 +1,432 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
tokers commented on issue #3283:
URL: https://github.com/apache/apisix/issues/3283#issuecomment-760581240
> But we don't pass the option to it.
You mean APISIX or others?
https://github.com/apache/apisix/blob/master/apisix/core/config_etcd.lua#L502
tokers commented on issue #3211:
URL: https://github.com/apache/apisix/issues/3211#issuecomment-760580542
> @ElvinEfendi Very insightful, thanks
>
> @moonming @membphis
> Reviewing [nginx's random
hnlq715 edited a comment on issue #3211:
URL: https://github.com/apache/apisix/issues/3211#issuecomment-760578226
@ElvinEfendi Very insightful, thanks
@moonming @membphis
Reviewing [nginx's random
spacewander commented on issue #3283:
URL: https://github.com/apache/apisix/issues/3283#issuecomment-760578300
@starsz
It would be great. We can do it in a separate PR.
This is an automated message from the Apache Git
1 - 100 of 219 matches
Mail list logo
