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.

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

Fix bogus http requests on diffs #13760

Merged
merged 1 commit into from
Nov 30, 2020
Merged

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