yiiviet / yii2-payment Goto Github PK

Xampp mình cài php 7.2 không cài được.

Hiện tại do chúng ta phân biệt ký tự hoa thường (sensitive) nên đôi khi không khớp với kiểu dữ liệu gửi từ Bảo Kim.

Cách giải quyết: hổ trợ thêm thuộc tính caseSensitive ở lớp \yiiviet\payment\HmacDataSignature

Hổ trợ tích hợp vtc pay ở phiên bản tiếp theo.

Tại dòng yiiviet\payment\vtcpay\VerifiedData #64 thực hiện lọc các dữ liệu gửi từ VTCPay nhưng nếu như dữ liệu gửi từ họ là rỗng thì có thể xảy ra việc xác minh chữ ký không đúng.

Các property của client hiện không kiểm tra lỗi bắt buộc khi cấu hình hệ thống nên khi xảy ra lỗi sẽ khó trace. Ở version tiếp theo nên kiểm tra và throw exception.

Hổ trợ tích hợp MOMO ở phiên bản tiếp theo.

Đối với các hệ thống sử dụng multi client thì ta nên hổ trợ thiết lập clientId tại lớp yiiviet\payment\VerifyFilter chứ không nên chỉ sử dụng clientId mặc định.

Hiện tại chúng ta phải tắt thủ công thuộc tính enableCsrfValidation trong controller, việc này khá bất tiện do tất cả cổng thanh toán đều phải tắt thuộc tính này (đối với post request). Nên ta nên thêm thuộc tính tự động disable sau khi kiểm tra thành công.

Khi tạo yêu cầu thanh toán với phương thức pro thì không cần checksum nhưng hiện tại hệ thống vẫn yêu cầu!

Hiện việc hổ trợ đơn giản hóa các prefix vnp_ hay vpc_ làm cho việc khó hiểu xảy ra khi so sánh các tham trị với tài liệu của 2 cổng thanh toán trên nên từ version 1.1.0 nên xóa bỏ magic properties.

Nhằm đơn giản hóa việc verify ipn mình đề xuất nên hổ trợ thêm action IPN để hổ trợ thao tác kiểm tra. Action chỉ việc xử lý thao tác nghiệp vụ thông qua successCallback hay errorCallback.

• Hổ trợ lấy danh sách ngân hàng phục vụ cho việc xây dựng widget hổ trợ lấy id của ngân hàng truyền vào phương thức purchase .

• Hổ trợ widget lấy danh sách ngân hàng bao gồm:
  • List
  • Dropdown list

Hiện phương thức verifyRequest của OnePay đang không xác minh được tính hợp lệ của chữ ký dữ liệu với params phản hồi trong môi trường sandbox:

Array
(
    [vpc_AdditionData] => 970425
    [vpc_Amount] => 500000000
    [vpc_Command] => pay
    [vpc_CurrencyCode] => VND
    [vpc_Locale] => vn
    [vpc_MerchTxnRef] => 1546483863
    [vpc_Merchant] => ONEPAY
    [vpc_OrderInfo] => 1546483863
    [vpc_TransactionNo] => 1681946
    [vpc_TxnResponseCode] => 0
    [vpc_Version] => 2
    [vpc_SecureHash] => 230EB9967004BBC35308BF95BCAAD2D28ADCFD3C928FC5C928BC36C44DE30615
)

Hiện đã có bank widget #1 cung cấp danh sách ngân hàng nhưng lại thiếu validator để kiểm tra tính hợp lệ của mã ngân hàng khi gửi lên. Nên ta nên hổ trợ thêm lớp validator để đảm nhiệm vấn đề này.