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.

Sign up for GitHub

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

Jump to bottom

[BUG] Swagger doc registered appname use contextPath #4988

Open
1 task done
GOODBOY008 opened this issue Aug 10, 2023 · 1 comment · May be fixed by #4989
Open
1 task done

[BUG] Swagger doc registered appname use contextPath #4988

GOODBOY008 opened this issue Aug 10, 2023 · 1 comment · May be fixed by #4989
Labels
type: bug Something isn't working

Comments

@GOODBOY008
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

image

Expected Behavior

No response

Steps To Reproduce

No response

Environment

ShenYu version(s):

Debug logs

No response

Anything else?

No response
@GOODBOY008 GOODBOY008 added the type: bug Something isn't working label Aug 10, 2023
GOODBOY008 added a commit to GOODBOY008/shenyu that referenced this issue Aug 10, 2023
@GOODBOY008
[ISSUE apache#4988] Fix swagger doc registered appName use contextPath
9c9c713
GOODBOY008 added a commit to GOODBOY008/shenyu that referenced this issue Aug 10, 2023
@GOODBOY008
[ISSUE apache#4988] Fix swagger doc registered appName use contextPath
0f83b53
@GOODBOY008 GOODBOY008 linked a pull request Aug 10, 2023 that will close this issue
Open
GOODBOY008 added a commit to GOODBOY008/shenyu that referenced this issue Aug 10, 2023
@GOODBOY008
[ISSUE apache#4988] Fix swagger doc registered appName use contextPath
e712d58
@lianjunwei
Copy link
Contributor

lianjunwei commented Aug 13, 2023 •
edited

Currently, the apidoc generated by Shenyu client annotations also uses contextpath as appname (that is, the tagName field of the tag table). In fact, at present, the apidoc function cannot get the appName, so the contextpath is temporarily used as the appName, and this function will be added later. In addition, the appname field should also be added to the database table.Soon, I'll start a discussion on how to fix this feature, and I'll notify you to participate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ISSUE #4988] Fix swagger doc registered appName use contextPath GOODBOY008/shenyu
2 participants
@GOODBOY008 @lianjunwei