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

Duplicate post request #6189

Closed
2 of 7 tasks
yasuokav opened this issue Feb 26, 2019 · 4 comments
Closed
2 of 7 tasks

Duplicate post request #6189

yasuokav opened this issue Feb 26, 2019 · 4 comments
Labels
topic/ui Change the appearance of the Gitea UI type/proposal The new feature has not been accepted yet but needs to be discussed first.

Comments

@yasuokav
Copy link
Contributor

yasuokav commented Feb 26, 2019

  • Gitea version (or commit ref): 594f591
  • Git version: 2.20.1
  • Operating system: macOS
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:
Start migrating repository "repo"
[Macaron] 2019-02-26 10:44:13: Started POST /repo/migrate for [::1]

... (migrating other repositories)

[Macaron] 2019-02-26 10:49:40: Started POST /repo/migrate for [::1]

...

2019/02/26 10:49:40 [I] [SQL] ROLL BACK
2019/02/26 10:49:40 [...routers/repo/repo.go:146 handleCreateError()] [E] MigratePost: repository already exists [uname: Mirror, name: repo]
2019/02/26 10:49:40 [D] Template: status/500

...

2019/02/26 10:50:10 [I] [SQL] COMMIT
2019/02/26 10:50:10 [T] Repository migrated [1285]: Mirror/repo
[Macaron] 2019-02-26 10:50:10: Completed POST /repo/migrate 302 Found in 5m56.741603459s

Description

Sometimes a strange situation happened to me when I migrating a repository

The post request of migrating will be sent twice and caused the error “repository already exists”.

Does anyone have any ideas?

http://geek.starbean.net/?p=393
This article could be help

Screenshots

@techknowlogick
Copy link
Member

I'm able to replicate this by clicking the migrate button multiple times, perhaps when buttons such as that are clicked they could then be disabled.

@techknowlogick techknowlogick added type/proposal The new feature has not been accepted yet but needs to be discussed first. topic/ui Change the appearance of the Gitea UI labels Feb 27, 2019
@yasuokav
Copy link
Contributor Author

yasuokav commented Feb 28, 2019

I'm able to replicate this by clicking the migrate button multiple times, perhaps when buttons such as that are clicked they could then be disabled.

This is another problem because it will immediately send the POST request twice and display a 500 error page instead of sending the second POST request after a few minutes.

@yasuokav
Copy link
Contributor Author

This issue can be closed after #6200 is merged.

@yasuokav
Copy link
Contributor Author

I assume it has been fixed.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic/ui Change the appearance of the Gitea UI type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

No branches or pull requests

2 participants