Skip to content

Commit fe04b8f

Browse files
committed
fix: remove Laravel 7 support due to compatibility issues
- Remove Laravel 7 from test matrix - Update documentation for Laravel 8-12 support only - Simplify PHP test code by removing Laravel 7 special case - Laravel 7 has different Encrypter initialization requirements
1 parent 2cb65ca commit fe04b8f

File tree

3 files changed

+6
-13
lines changed

3 files changed

+6
-13
lines changed

.github/workflows/test.yml

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -75,15 +75,15 @@ jobs:
7575
strategy:
7676
fail-fast: false
7777
matrix:
78-
laravel: [7, 8, 9, 10, 11, 12]
78+
laravel: [8, 9, 10, 11, 12]
7979

8080
steps:
8181
- uses: actions/checkout@v4
8282

8383
- name: Setup PHP
8484
uses: shivammathur/setup-php@v2
8585
with:
86-
php-version: ${{ matrix.laravel == 7 && '7.4' || matrix.laravel == 8 && '8.0' || matrix.laravel == 9 && '8.1' || matrix.laravel >= 10 && '8.3' }}
86+
php-version: ${{ matrix.laravel == 8 && '8.0' || matrix.laravel == 9 && '8.1' || matrix.laravel >= 10 && '8.3' }}
8787
extensions: mbstring, xml, json
8888

8989
- name: Setup Node.js
@@ -109,13 +109,7 @@ jobs:
109109
require 'vendor/autoload.php';
110110
111111
$key = base64_decode(getenv('TEST_KEY'));
112-
// Laravel 7 requires key to be passed differently
113-
$laravel_version = getenv('LARAVEL_VERSION');
114-
if ($laravel_version == '7') {
115-
$encrypter = new \Illuminate\Encryption\Encrypter($key);
116-
} else {
117-
$encrypter = new \Illuminate\Encryption\Encrypter($key, 'aes-256-cbc');
118-
}
112+
$encrypter = new \Illuminate\Encryption\Encrypter($key, 'aes-256-cbc');
119113
120114
// All test cases - verschiedene Datentypen!
121115
$testCases = [

README.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Laravel-compatible encryption/decryption for Node.js. Fully compatible with Lara
88

99
## Features
1010

11-
- ✅ 100% Laravel compatible (7.x - 12.x)
11+
- ✅ 100% Laravel compatible (8.x - 12.x)
1212
- ✅ Zero configuration (auto-detects `APP_KEY`)
1313
- ✅ AES-256-CBC encryption with HMAC-SHA256
1414
- ✅ No dependencies (optional `php-serialize` for complex objects)
@@ -114,7 +114,6 @@ app.post('/decrypt', (req, res) => {
114114
| 10.x | 16+ | 8.1+ ||
115115
| 9.x | 14+ | 8.0+ ||
116116
| 8.x | 14+ | 7.3+ ||
117-
| 7.x | 14+ | 7.2.5+ ||
118117

119118
## License
120119

src/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@ const crypto = require('crypto');
22

33
/**
44
* Laravel-compatible Encryption class for Node.js
5-
* Full compatibility with Laravel 7.x through 12.x
5+
* Full compatibility with Laravel 8.x through 12.x
66
*
77
* Tested with:
8-
* - Laravel 7.x, 8.x, 9.x, 10.x, 11.x, 12.x
8+
* - Laravel 8.x, 9.x, 10.x, 11.x, 12.x
99
* - Custom/legacy implementations
1010
* - Auto-detection of MAC calculation methods
1111
*

0 commit comments

Comments
 (0)