Project

General

Profile

Actions

Defect #7307

closed

HTTP 500 error on query for empty revision

Added by Etienne Massip about 13 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Toshi MARUYAMA
Category:
SCM
Target version:
Start date:
2011-01-12
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

On revision page, you can change the revision id then click on the 'Ok' button to get to the new revision page.

If you remove the revision id field content and leave it blank then you'll get a HTTP 500 error.

Additional question (UX) : why is the 'Ok' button displayed on the revision page and not on the repository page ?

Tested with a SVN repo.


Files

7307-rev-empty.diff (581 Bytes) 7307-rev-empty.diff Toshi MARUYAMA, 2011-01-12 11:01

Related issues

Has duplicate Redmine - Defect #7332: Defect #7307 doesn't appear to be fixedClosedToshi MARUYAMA2011-01-14

Actions
Actions #1

Updated by Toshi MARUYAMA about 13 years ago

This is ad hoc patch.

Actions #2

Updated by Gloriaer smith about 13 years ago

  • File leanspawithacai.jpg added
  • Assignee set to Maxim Krušina
Actions #3

Updated by Toshi MARUYAMA about 13 years ago

  • File deleted (leanspawithacai.jpg)
Actions #4

Updated by Toshi MARUYAMA about 13 years ago

  • Assignee deleted (Maxim Krušina)
Actions #5

Updated by Toshi MARUYAMA about 13 years ago

Etienne Massip wrote:

Additional question (UX) : why is the 'Ok' button displayed on the revision page and not on the repository page ?

Git shows branches and tags list box.
https://projects.kde.org/projects/extragear/kdevelop/kdevelop/repository

But, I don't know the reason of no 'Ok' button.

Actions #6

Updated by Etienne Massip about 13 years ago

Toshi MARUYAMA wrote:

This is ad hoc patch.

Do you want me to test it ?

Actions #7

Updated by Toshi MARUYAMA about 13 years ago

Etienne Massip wrote:

Toshi MARUYAMA wrote:

This is ad hoc patch.

Do you want me to test it ?

I tested in Subversion.
I will research other SCMs.

Actions #8

Updated by Toshi MARUYAMA about 13 years ago

  • Assignee set to Toshi MARUYAMA
Actions #9

Updated by Toshi MARUYAMA about 13 years ago

  • Status changed from New to Closed
  • Target version set to 1.1.1
  • Resolution set to Fixed

Fixed in trunk by r4703, r4704 and in 1.1 by r4707.

Actions #10

Updated by Ве Fio about 13 years ago

Hmm. I'm using 4707 right now, and the issue doesn't seem fixed for me. I click the repository tab, click view all revisions, then in the "Revision: <input box> <ok button>" I just click the ok button without entering anything, and up pops a 500 error.

Actions #11

Updated by Jean-Philippe Lang about 13 years ago

  • Status changed from Closed to Reopened

r4707 will return a 500 status code indeed. Maybe we could simply redirect to the revisions list when no revision parameter is given.

Actions #12

Updated by Etienne Massip about 13 years ago

I'm running r4709 and I get the same redmine 500 error I would get if I typed in a revision number which would not exist.

I don't have the Internal Server Error HTTP 500 any more (with svn, at least) so, as for myself, this issue is resolved.

Actions #13

Updated by Jean-Philippe Lang about 13 years ago

Yes, it's a user friendly 500 error but still, it's a 500 error response (meaning Internal Server Error) and this is not really appropriate. I think we should:
  • response with a 404 when the revision that was entered was not found
  • do a redirect to the revision list when no revision parameter is given
Actions #14

Updated by Etienne Massip about 13 years ago

Jean-Philippe Lang wrote:

Yes, it's a user friendly 500 error but still, it's a 500 error response (meaning Internal Server Error) and this is not really appropriate. I think we should:
  • response with a 404 when the revision that was entered was not found
  • do a redirect to the revision list when no revision parameter is given

Sure, both looks like a (far for 404) better behavior.

Actions #15

Updated by Jean-Philippe Lang about 13 years ago

500 error changed to 404 in r4711.

Actions #16

Updated by Toshi MARUYAMA about 13 years ago

  • Status changed from Reopened to Closed

Fixed in trunk by from r4712 to r4714 and in 1.1 by from r4715 to r4717.

Actions

Also available in: Atom PDF