mirror of
https://github.com/ether/etherpad-lite.git
synced 2025-01-19 14:13:34 +01:00
Fix/rate limit in test (#5931)
* build(deps): bump express-rate-limit from 6.11.1 to 7.0.0 in /src Bumps [express-rate-limit](https://github.com/express-rate-limit/express-rate-limit) from 6.11.1 to 7.0.0. - [Release notes](https://github.com/express-rate-limit/express-rate-limit/releases) - [Changelog](https://github.com/express-rate-limit/express-rate-limit/blob/main/changelog.md) - [Commits](https://github.com/express-rate-limit/express-rate-limit/compare/v6.11.1...v7.0.0) --- updated-dependencies: - dependency-name: express-rate-limit dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> * Increase exportratelimit to high number for tests. * Fixed rate limiting in frontend tests. * Also do this for the run without plugins. --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This commit is contained in:
parent
80cd1b6127
commit
1b01f01ca3
3 changed files with 16 additions and 4 deletions
8
.github/workflows/frontend-admin-tests.yml
vendored
8
.github/workflows/frontend-admin-tests.yml
vendored
|
@ -67,8 +67,12 @@ jobs:
|
||||||
name: Write custom settings.json that enables the Admin UI tests
|
name: Write custom settings.json that enables the Admin UI tests
|
||||||
run: "sed -i 's/\"enableAdminUITests\": false/\"enableAdminUITests\": true,\\n\"users\":{\"admin\":{\"password\":\"changeme\",\"is_admin\":true}}/' settings.json"
|
run: "sed -i 's/\"enableAdminUITests\": false/\"enableAdminUITests\": true,\\n\"users\":{\"admin\":{\"password\":\"changeme\",\"is_admin\":true}}/' settings.json"
|
||||||
-
|
-
|
||||||
name: increase maxHttpBufferSize
|
name: Disable import/export rate limiting
|
||||||
run: "sed -i 's/\"maxHttpBufferSize\": 10000/\"maxHttpBufferSize\": 100000/' settings.json"
|
run: |
|
||||||
|
sed -e '/^ *"importExportRateLimiting":/,/^ *\}/ s/"max":.*/"max": 1000000/' -i settings.json
|
||||||
|
-
|
||||||
|
name: Increase express-rate-limit
|
||||||
|
run:
|
||||||
-
|
-
|
||||||
name: Remove standard frontend test files, so only admin tests are run
|
name: Remove standard frontend test files, so only admin tests are run
|
||||||
run: mv src/tests/frontend/specs/* /tmp && mv /tmp/admin*.js src/tests/frontend/specs
|
run: mv src/tests/frontend/specs/* /tmp && mv /tmp/admin*.js src/tests/frontend/specs
|
||||||
|
|
4
.github/workflows/frontend-tests.yml
vendored
4
.github/workflows/frontend-tests.yml
vendored
|
@ -43,7 +43,7 @@ jobs:
|
||||||
-
|
-
|
||||||
name: Disable import/export rate limiting
|
name: Disable import/export rate limiting
|
||||||
run: |
|
run: |
|
||||||
sed -e '/^ *"importExportRateLimiting":/,/^ *\}/ s/"max":.*/"max": 0/' -i settings.json
|
sed -e '/^ *"importExportRateLimiting":/,/^ *\}/ s/"max":.*/"max": 100000000/' -i settings.json
|
||||||
-
|
-
|
||||||
uses: saucelabs/sauce-connect-action@v2.3.4
|
uses: saucelabs/sauce-connect-action@v2.3.4
|
||||||
with:
|
with:
|
||||||
|
@ -127,7 +127,7 @@ jobs:
|
||||||
-
|
-
|
||||||
name: Disable import/export rate limiting
|
name: Disable import/export rate limiting
|
||||||
run: |
|
run: |
|
||||||
sed -e '/^ *"importExportRateLimiting":/,/^ *\}/ s/"max":.*/"max": 0/' -i settings.json
|
sed -e '/^ *"importExportRateLimiting":/,/^ *\}/ s/"max":.*/"max": 1000000/' -i settings.json
|
||||||
# XXX we should probably run all tests, because plugins could effect their results
|
# XXX we should probably run all tests, because plugins could effect their results
|
||||||
-
|
-
|
||||||
name: Remove standard frontend test files, so only plugin tests are run
|
name: Remove standard frontend test files, so only plugin tests are run
|
||||||
|
|
|
@ -1,3 +1,11 @@
|
||||||
|
# 1.9.3
|
||||||
|
|
||||||
|
### Compability changes
|
||||||
|
|
||||||
|
* express-rate-limit has been bumped to 7.0.0: This involves the breaking change that "max: 0"
|
||||||
|
in the importExportRateLimiting is set to always trigger. So set it to your desired value.
|
||||||
|
If you haven't changed that value in the settings.json you are all set.
|
||||||
|
|
||||||
# 1.9.2
|
# 1.9.2
|
||||||
|
|
||||||
### Notable enhancements and fixes
|
### Notable enhancements and fixes
|
||||||
|
|
Loading…
Reference in a new issue