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

Adjust suffix in server to allow redirection when needed #2931

Merged
merged 1 commit into from
Oct 5, 2021

Conversation

avaris
Copy link
Member

@avaris avaris commented Oct 4, 2021

Folders without index.html has to be redirected (/foo -> /foo/) for
directory listing to work properly. Skip '/' suffix if original path
does not have it so that base class can return a redirect.

Pull Request Checklist

Resolves: #2841

  • Ensured tests pass and (if applicable) updated functional test output
  • Conformed to code style guidelines by running appropriate linting tools
  • Added tests for changed code
  • N/A Updated documentation for changed code

Folders without index.html has to be redirected (/foo -> /foo/) for
directory listing to work properly. Skip '/' suffix if original path
does not have it so that base class can return a redirect.
@justinmayer
Copy link
Member

Many thanks for the fix, @avaris! 🏅

@justinmayer justinmayer merged commit 324fcef into getpelican:master Oct 5, 2021
@avaris avaris deleted the server-list-files branch October 5, 2021 19:40
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

Successfully merging this pull request may close these issues.

drafts index page renders with incorrect links to draft articles if it doesnt have a trailing /
2 participants