Skip to content

Browsers without SubmitEvent.submitter support don't work with form-fetch-action mechanism well #28319

Closed
@wolfbeast

Description

@wolfbeast

Description

I recently updated to 1.21.0 and am running into an issue now that I cannot close or re-open pull requests from the discussion page buttons at the bottom (neither with or without comment). I did not have this issue on our last version (1.19.4).

I am still able to open and close pull requests from the pull request listing (checking the boxes then clicking the close button).

I have not seen any error message be thrown in the UI.

Gitea Version

1.21.0

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

No response

Screenshots

FireShot Pro Screen Capture #396 - '#1 - Update test md - bug-test-repo - Gitea_ Git with a cup of tea' - try gitea io_wolfbeast71_bug-test-repo_pulls_1

Git Version

No response

Operating System

CentOS 7

How are you running Gitea?

Binary download from gitea distr. natively installed (command-line/service)

Database

MySQL/MariaDB

Activity

lng2020

lng2020 commented on Dec 4, 2023

@lng2020
Member

What do you mean by Doesn't work? You can't click on it, or after clicking on it, nothing changes.

added
issue/needs-feedbackFor bugs, we need more details. For features, the feature must be described in more detail
on Dec 5, 2023
wolfbeast

wolfbeast commented on Dec 6, 2023

@wolfbeast
Author

Sorry if I wasn't clear. I can click on it, the page refreshes, but nothing changes in the status of the pull request.

ghost

ghost commented on Dec 11, 2023

@ghost

I'm having the same/similar issue. It's a Draft PR with one dependency.

You need to close all issues blocking this pull request before you can merge it

Tried closing via the PR listing, but then it says

Cannot batch close issues that you choose, because issue #115487 still has open dependencies

This seems backwards, a dependency should not prevent me from closing the PR.

wolfbeast

wolfbeast commented on Dec 11, 2023

@wolfbeast
Author

I'm having the same/similar issue.

I don't think that's related. This issue occurs for me regardless of dependencies and does not throw any errors. It just reloads the page (as if the command would be processed) but the status of the PR does not change, no error given.

removed
issue/needs-feedbackFor bugs, we need more details. For features, the feature must be described in more detail
on Dec 11, 2023
wxiaoguang

wxiaoguang commented on Dec 11, 2023

@wxiaoguang
Contributor

I'm having the same/similar issue.

I don't think that's related. This issue occurs for me regardless of dependencies and does not throw any errors. It just reloads the page (as if the command would be processed) but the status of the PR does not change, no error given.

Quote your issue:

Can you reproduce the bug on the Gitea demo site?

Yes

How to?

added
issue/needs-feedbackFor bugs, we need more details. For features, the feature must be described in more detail
and removed on Dec 12, 2023
wolfbeast

wolfbeast commented on Dec 12, 2023

@wolfbeast
Author

How to?

  1. edit a file in a repo on a separate branch
  2. create a PR from the branch comparison
  3. attempt to close the PR from the comment tab on the PR (see screenshot)
wxiaoguang

wxiaoguang commented on Dec 12, 2023

@wxiaoguang
Contributor

On try.gitea.io https://try.gitea.io/wxiaoguang/test/pulls/35

1 & 2: edit a file in a repo on a separate branch, create a PR from the branch comparison

image

image

  1. attempt to close the PR from the comment tab on the PR (see screenshot)

image

Everything seems fine.

wolfbeast

wolfbeast commented on Dec 12, 2023

@wolfbeast
Author

I'm actually seeing the same simply with an open issue. can't close it with the button. I'll see if I can get a log.

wxiaoguang

wxiaoguang commented on Dec 12, 2023

@wxiaoguang
Contributor

Double confirm: are you sure you can reproduce it on try.gitea.io? If yes, is it browser related? If no, is it custom-template related?

wolfbeast

wolfbeast commented on Dec 12, 2023

@wolfbeast
Author

I can't see anything in our gitea log. Nothing in the browser console either. no errors on the page.

The behavior when checking the network request seems to be the following:

  1. POST request to "comments"
  2. response is a redirect: to the issue/pr URL.
  3. POST request to /-/fetch-redirect URL.
  4. response is 303 to the issue/PR page
  5. GET request to the issue/PR page
  6. response is the issue/PR page
    with further loading of the page

has anything in that respect changed since 1.19?

35 remaining items

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    issue/needs-feedbackFor bugs, we need more details. For features, the feature must be described in more detail

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @wxiaoguang@wolfbeast@delvh@lng2020

      Issue actions

        Browsers without SubmitEvent.submitter support don't work with form-fetch-action mechanism well · Issue #28319 · go-gitea/gitea