Skip to content

Conversation

@wxiaoguang
Copy link
Contributor

The submitted tests in the patch for the XSS fix is not right.

To test, it should test "what should happen", but not "what doesn't exist" or "what is processed/decoded".

@wxiaoguang wxiaoguang added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Feb 24, 2024
@wxiaoguang wxiaoguang added this to the 1.21.7 milestone Feb 24, 2024
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 24, 2024
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Feb 24, 2024
@wxiaoguang wxiaoguang linked an issue Feb 24, 2024 that may be closed by this pull request
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Feb 24, 2024
@lunny lunny merged commit 7274357 into go-gitea:release/v1.21 Feb 24, 2024
@wxiaoguang wxiaoguang deleted the fix-test branch February 24, 2024 07:28
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky test: TestXSSWikiLastCommitInfo

4 participants