Skip to content

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Nov 30, 2020

Not all .blob-excerpt elements have these data attributes resulting in bogus http request when expanding a diff and clicking into the expanded area. This prevents those.

Should backport to 1.13.

Fixes: #13759

The .blob-excerpt elements don't have these data attributes in some
cases resulting in bogus http request when expanding a diff and clicking
into the expanded area. This prevents those.

Should backport to 1.13.

Fixes: go-gitea#13759
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Nov 30, 2020
@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 Nov 30, 2020
@techknowlogick techknowlogick added this to the 1.14.0 milestone Nov 30, 2020
@techknowlogick techknowlogick merged commit 21cf205 into go-gitea:master Nov 30, 2020
@silverwind silverwind deleted the fix401 branch November 30, 2020 17:49
silverwind added a commit to silverwind/gitea that referenced this pull request Nov 30, 2020
The .blob-excerpt elements don't have these data attributes in some
cases resulting in bogus http request when expanding a diff and clicking
into the expanded area. This prevents those.

Should backport to 1.13.

Fixes: go-gitea#13759
techknowlogick pushed a commit that referenced this pull request Nov 30, 2020
The .blob-excerpt elements don't have these data attributes in some
cases resulting in bogus http request when expanding a diff and clicking
into the expanded area. This prevents those.

Should backport to 1.13.

Fixes: #13759
@techknowlogick techknowlogick added the backport/done All backports for this PR have been created label Dec 2, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Jan 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JS requests causing HTTP basic auth popup
5 participants