pagure

#1742 Dont return 404 for empty files
Merged by pingou. Opened by cep.
cep/pagure fix_error_404_for_empty_files  into  master

Download 1742.patch
no initial comment

:information_source: Not up for review yet.

It works, but I'm still testing to see if I missed anything. Comments welcome :)

Could we have a way of finding if the file is empty or absent?

rebased

Nice and easy :)

Took an awful lot of navigating the maze though. But was worth it :)
Give me a few to update the tests.

1 new commit added

  • Add tests for displaying empty files

rebased

PR is up for review now! :boom:

That's one empty line too many to please pep8 :)

One tiny comment but this looks good, I'll test it locally before merging though, thanks! :)

2 new commits added

  • Add tests for displaying empty files
  • Dont return 404 when viewing empty files

Needs a rebase :)

rebased

And done! :)

And merging! :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.