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

Gollum::Wiki.pages hangs indefiniently #55

Closed
henrikh opened this issue Oct 5, 2010 · 7 comments
Closed

Gollum::Wiki.pages hangs indefiniently #55

henrikh opened this issue Oct 5, 2010 · 7 comments

Comments

@henrikh
Copy link
Contributor

henrikh commented Oct 5, 2010

The method pages hangs indefiniently when used on my computer. Using Git Bisect I've tracked the error to the commit http://github.com/github/gollum/commit/e37a8b9da473cb0301d40be28b72e15b0bee31d1

To reproduce:

  • Open a Gollum "repo" with ruby [pathpath] --irc
  • Enter wiki.pages
  • Now it should hang there indefiniently

On any previous commit, the method would return a list of all pages.

I'm interested in knowing if anyone else can reproduce this bug.

@technoweenie
Copy link
Contributor

How many pages is it returning? It's calling #title too, so it's parsing the title out of the formatted page. It's pretty important for the page index, otherwise you get pages out of order. Though, it's probably best to remove that code because basic gollum repos don't cache the formatted output. We cache all of that on GitHub, so I never noticed any performance hits.

@henrikh
Copy link
Contributor Author

henrikh commented Oct 5, 2010

My computer might be slow (it is), but it takes about one minute to serve the list of pages.

@technoweenie
Copy link
Contributor

For how many pages?

@henrikh
Copy link
Contributor Author

henrikh commented Oct 5, 2010

25 pages

@henrikh
Copy link
Contributor Author

henrikh commented Oct 5, 2010

I've "fixed" it on my branch pages_sorted_by_title

@technoweenie
Copy link
Contributor

Cool. I'll probably just remove the code and leave it as an exercise of the user to sort stuff. I hear ruby can do that pretty easily :) Thanks for the report!

@technoweenie
Copy link
Contributor

Henrikh: I just reverted the commit: dd7dddd

dometto pushed a commit that referenced this issue Jan 8, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants