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

Fix 413 request entity too large #55

Merged
merged 2 commits into from Aug 23, 2018

Conversation

Projects
None yet
4 participants
@zamentur
Copy link
Contributor

commented Jun 30, 2018

This fix an issue when you try to import wallabag articles:

https://forum.yunohost.org/t/wallabag2-413-request-entity-too-large/5124https://forum.yunohost.org/t/wallabag2-413-request-entity-too-large/5124

Problem

  • Description of why you made this PR

Solution

  • And how you fix that

PR Status

  • Code finished.
  • Tested with Package_check.
  • Fix or enhancement tested.
  • Upgrade from last version tested.
  • Can be reviewed and tested.

Validation


Minor decision

  • Upgrade previous version :
  • Code review : JimboJoe
  • Approval (LGTM) : JimboJoe
  • Approval (LGTM) : Josué
  • CI succeeded :
    Build Status

When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.

zamentur added some commits Jun 30, 2018

@zamentur zamentur changed the base branch from master to testing Jun 30, 2018

@JimboJoe
Copy link
Contributor

left a comment

LGTM and code review OK 👍

@JimboJoe JimboJoe requested a review from YunoHost-Apps/apps-group Aug 4, 2018

@Josue-T

Josue-T approved these changes Aug 4, 2018

@Josue-T

This comment has been minimized.

Copy link

commented Aug 4, 2018

Will be merged in 3 days.

@lapineige lapineige merged commit bf1431b into testing Aug 23, 2018

@maniackcrudelis maniackcrudelis deleted the fix-413-request-entity-too-large branch Aug 23, 2018

@lapineige lapineige referenced this pull request Aug 29, 2018

Merged

Testing #56

3 of 8 tasks complete

lapineige added a commit that referenced this pull request Dec 17, 2018

Testing (#56)
* Fix 413 request entity too large (#55)

* [fix] 413 Request Entity Too Large

* Remove ini file for php (#57)

Have a look to YunoHost-Apps/nextcloud_ynh#138 for more information

lapineige added a commit that referenced this pull request May 8, 2019

Add Fail2ban support (#65)
* Readme: Add welcome UI screenshoot 

It's always nice to have a quick look at the UI of the software you're going to install :)

* Testing  (#56)

* Fix 413 request entity too large (#55)

* [fix] 413 Request Entity Too Large

* Remove ini file for php (#57)

Have a look to YunoHost-Apps/nextcloud_ynh#138 for more information

* Add fail2ban config

* Add Fail2ban config removal helper

* Enhance fail2ban config look

* Add fail2ban backup

* Restore fail2ban config

* Add fail2ban config during upgrade

To make sure any older version will get fail2ban support

* Update minimum version to 3.5

This is needed for fail2ban helpers

* Add fail2ban info

* Fail2ban: Fix missing log file during install

* Fix feil2ban regex

* Fix fail2ban regex - 2

* Use ynh_systemd_action

* Use long getopts arguments

* Fix duplicated comment and remove blank space

* Fix fail2ban regex in upgrade script

* Improve regex - install

This allow empty username (not possible, but may still block some extra brute force spammers) and username with spaces

* Improve regex - upgrade

This allow empty username (not possible, but may still block some extra brute force spammers) and username with spaces

* Fix missing log file for fail2ban

* Indentation and variable usage

@maniackcrudelis maniackcrudelis referenced this pull request May 13, 2019

Merged

Testing #70

7 of 8 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.