Gollum: Page deletion in Overview doesn't work if base-path option is given

Created on 13 Dec 2020  路  8Comments  路  Source: gollum/gollum

Describe the bug
Page deletion doesn't work if you attempt this action in Overview and gollum is run with base-path option. It doesn't return any error but the deletion commit ends up empty.

To Reproduce
Run gollum with base-path option: gollum --base-path wiki

Expected behavior
Deletion should work

Environment Info

$ gollum --versions
Gollum 5.1.2
Running on: x86_64-linux-gnu with Ruby version 2.7.0
Using:
rugged 0.99.0
gollum-rugged_adapter 1.0
gollum-lib 5.0.6
Markdown rendering gem: kramdown
Other renderering gems:
none

All 8 comments

Worth to mention that the same bug occurs if I don't specify the base-path but use an auth middleware I've written: https://github.com/ViChyavIn/gollum-firebase-auth

Thanks! That means it's almost definitely the same issue as #1620.

The patch in https://github.com/gollum/gollum/pull/1639 fixes the issue with base path. @ViChyavIn could you perhaps check whether it also solves the problem without base path, but with the auth middleware?

Yes, it fixes both cases! But still breaks if I specify template-dir :/

Nevermind, the problem is probably on my side: I replaced the templates with the default versions, leaving templates-dir option, and deletion worked. Thank you!

Ah, you changed overview.mustache but I still had the old version. Replaced it with the new one and everything started working.

Yeah, sorry! My initial fix broke the links to the pages in the Overview. Glad you got it working, and thanks for the feedback!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jb2cool picture jb2cool  路  3Comments

thorlucas picture thorlucas  路  4Comments

hannahwong picture hannahwong  路  10Comments

x-yuri picture x-yuri  路  5Comments

waltharius picture waltharius  路  7Comments