apache / apache/shenyu

[BUG] AppAuthServiceImpl.updateDetail path appName uses first old path's appName (cross-link)

Open
#6,548 1 comment 0 reactions 0 assignees View on GitHub
admin priority: medium type: bug
Dominant language
Java
Stars
8.8k
Forks
3.1k
Avg merge
7d 1h
Merged PRs (30d)
85

Description

## Description
See the dedicated issue for `updateDetail`'s path appName handling — this entry is a tracking stub; the full detail is in the audit doc.

## Location
```
shenyu-admin/.../AppAuthServiceImpl.java:197-204
```

## Impact
Path appName silently lost/emptied.

## Suggested fix
Use `dto.getAppName()` per path.

## Related existing issue(s)
Covered by the updateDetail High issue above; grouped under Medium for visibility.

_Identified during the 2026-08-02 audit; full list in [`docs/issue-candidates-2026-08-02.md`](docs/issue-candidates-2026-08-02.md)._

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading shenyu-admin/.../AppAuthServiceImpl.java lines 197-204 and consult docs/issue-candidates-2026-08-02.md for the full updateDetail context. Trace how each path is processed and verify that the completed behavior preserves the appName associated with its own DTO rather than an earlier path.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.