Skip to content

Commit b629087

Browse files
committed
Adjust composer.json, readme and pipeline files. Wrap exception handler
around unserialize.
1 parent 10fac2d commit b629087

5 files changed

Lines changed: 76 additions & 1807 deletions

File tree

README.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,17 +5,17 @@ It can be implemented inside of other content management systems to provide fire
55

66
## How do I get set up? ##
77

8-
Due to library limitations and the way the firewall engine works, a few manual interactions are required. Since the firewall engine depends on the opis/closure and laravel/serializable-closure packages which both require different PHP versions, sticking to one PHP version is tricky.
8+
Since the firewall engine depends on the opis/closure and laravel/serializable-closure packages which both require different PHP versions, sticking to one PHP version is tricky.
99

10-
Therefore, in order to use this package, these libraries must exist in the following locations:
11-
* opis/closure at /vendor/closure/
12-
* laravel/serializable-closure at /vendor/serializable-closure/
10+
Therefore, in order to use this package, the composer install command must be executed with the --ignore-platform-reqs flag:
11+
12+
`composer install --ignore-platform-reqs`
1313

1414
## To-Do List ##
15+
- ✅ First initial concept and implementation
1516
- ⬜️ Unit tests
1617
- ⬜️ Test on many different environments and combinations with other plugins
1718
- ⬜️ Attach PHP CS Fixer
18-
- ✅ First initial concept and implementation
1919

2020
## Who do I talk to? ##
2121

bitbucket-pipelines.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
image: php:7.3
1+
image: php:7.4
22

33
pipelines:
44
default:
@@ -9,7 +9,7 @@ pipelines:
99
- apt-get update && apt-get install -qy git curl libzip-dev zip
1010
- docker-php-ext-install zip
1111
- curl -sS https://getcomposer.org/installer | php -- --install-dir=/usr/local/bin --filename=composer
12-
- composer install
12+
- composer install --ignore-platform-reqs
1313
- ./vendor/bin/phpunit tests
1414
caches:
1515
- composer

composer.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,9 @@
33
"description": "The firewall engine of Patchstack.",
44
"type": "library",
55
"require": {
6-
"php": ">=5.4"
6+
"php": ">=5.4",
7+
"opis/closure": "^3.6",
8+
"laravel/serializable-closure": "^1.1"
79
},
810
"license": "GNU AGPLv3",
911
"autoload": {

0 commit comments

Comments
 (0)