Automattic / Automattic/VIP-Coding-Standards
Custom IniSet Sniff to allow for certain safelisted options
- Ngôn ngữ chính
- PHP
- Star
- 261
- Fork
- 44
- Merge trung bình
- 19 phút
- Pull request đã merge (30 ngày)
- 1
Mô tả
## Describe the solution you'd like
VIPCS could benefit from having a custom IniSet sniff...similar to https://github.com/WordPress/WordPress-Coding-Standards/blob/41f5a9c66ff814863bc479fb52fd6cd1abc87e28/WordPress/Sniffs/PHP/IniSetSniff.php#L55-L65, but we want to be able to customize the whitelist property for the below values:
- session.cookie_httponly
- session.cookie_secure
- session.use_only_cookies
WP already manages sessions but we have some clients that want to control PHP sessions via custom code.
## What code should *not* be reported as a violation?
```
ini_set('session.cookie_httponly', true);
ini_set('session.cookie_secure', true);
ini_set('session.use_only_cookies', true);
```
## Additional context
https://github.com/WordPress/WordPress-Coding-Standards/issues/1993
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách so sánh triển khai được tham chiếu trong WordPress/Sniffs/PHP/IniSetSniff.php, đặc biệt là các dòng 55-65, với cấu trúc sniff của VIP-Coding-Standards. Xác minh whitelist được cấu hình như thế nào và xác định các bài kiểm thử sniff liên quan. Hoàn thành có nghĩa là ba tùy chọn session được liệt kê được chấp nhận, trong khi các lệnh gọi ini_set không được phép khác vẫn là các vi phạm.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- php
- Lĩnh vực
- tooling
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 48/100