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

No Error Reported when Closing Issue Failed due to Underlying Dependency #18236

Closed
CasperHK opened this issue Jan 11, 2022 · 8 comments
Closed
Labels
issue/duplicate The issue has already been reported. issue/regression Issue needs no code to be fixed, only a description on how to fix it yourself type/bug
Milestone

Comments

@CasperHK
Copy link

CasperHK commented Jan 11, 2022

Gitea Version

1.16.0

Git Version

No response

Operating System

Ubuntu

How are you running Gitea?

I host it by myself in a VM inside intranet.

Database

MySQL

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Description

I found an error when closing an issue. When I click on the Close button in the bottom comment section, the website get redirected to the same page but the issue is not yet closed. I cannot reproduce the same error on other issue post on the same site. I consider it may be a bug in Gitea.

Current Action

Currently, I just close the related post manually by setting the is_closed flag to 1 in the database, which means closed. However, the problem is that the person who closed the post does not record in the history.

Someone encountered the same problem before? And, how can you fix it? Please help.

Thank you.

Screenshots

No response

@zeripath
Copy link
Contributor

Why do we always have to repeat this:

Please read: https://docs.gitea.io/en-us/logging-configuration/#debugging-problems

There will almost certainly be something helpful to help further explain what you're seeing in the logs. Could you please give us some logs.

However, in this case we're very likely to need LOG_SQL=true here too.

So please could you set your logging to:

[database]
LOG_SQL = true ; Normally we don't want or need SQL logging for debug logs however, in this case it's likely to be helpful.

...

[log]
MODE = console
LEVEL = debug ; please set the level to debug when we are debugging a problem
ROUTER = console
COLORIZE = false ; this can be true if you can strip out the ansi coloring

@zeripath zeripath added the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label Jan 11, 2022
@Galaxy102
Copy link

I was able to reproduce this issue on my instance when trying to close an issue with dependencies. It appears you can't close an issue when there are other issues blocking it. There is no error message whatsoever, just nothing happens. I have the Logs attached.

It would be great if Gitea would send an alert stating the dependency.
gitea.log

@zeripath
Copy link
Contributor

It think this may be related to #17643

@zeripath
Copy link
Contributor

There should be an error reported - but it's not happening.

@zeripath zeripath removed the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label Jan 15, 2022
@zeripath zeripath changed the title Unable to Close Issue in Gitea No Error reported when closing issue fails due to underlying dependency Jan 15, 2022
@zeripath zeripath added this to the 1.16.0 milestone Jan 15, 2022
@zeripath zeripath added type/bug issue/regression Issue needs no code to be fixed, only a description on how to fix it yourself labels Jan 15, 2022
@zeripath
Copy link
Contributor

Yup this is a regression due to #17643 - we should be handling the dependency error in the callers.

@zeripath
Copy link
Contributor

Duplicate of #18223

@zeripath zeripath marked this as a duplicate of #18223 Jan 15, 2022
@zeripath zeripath added the issue/duplicate The issue has already been reported. label Jan 15, 2022
@CasperHK CasperHK changed the title No Error reported when closing issue fails due to underlying dependency No Error Reported when Closing Issue Failed due to Underlying Dependency Jan 16, 2022
@CasperHK
Copy link
Author

CasperHK commented Mar 1, 2022

Due to no error message shown to the user, user may directly thinking of whether it is a bug. This means if the dependence not yet closed, then the target issue will not be able to be closed too. Then, the error message should be quite important.

@zeripath
Copy link
Contributor

zeripath commented Mar 1, 2022

This is already fixed in 1.16+ by #18325. If you're seeing this problem in 1.16 you need to open a new issue and give us full details. We will not backport this to 1.15.

Commenting on duplicate issues which have been closed over a month ago and where the original issue already has been fixed by a PR is not helpful.

@go-gitea go-gitea locked as resolved and limited conversation to collaborators Mar 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/duplicate The issue has already been reported. issue/regression Issue needs no code to be fixed, only a description on how to fix it yourself type/bug
Projects
None yet
Development

No branches or pull requests

3 participants