search

yajra / laravel-datatables-html

Laravel DataTables HTML Builder Plugin
https://yajrabox.com/docs/laravel-datatables/html-installation
MIT License
272 stars 63 forks source link

fix: Mixed Content problem with updating minifiedAjax method and get current url based on http or https scheme #186

Closed huzairsaputra closed 1 year ago

huzairsaputra commented 1 year ago

Change url()->full() method to url()->current() to solve Mixed Content when using forceScheme('https') function in Laravel

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

yajra commented 1 year ago

This PR will revert https://github.com/yajra/laravel-datatables-html/pull/122. Need to confirm with other users before merging. There was also a comment to revert it https://github.com/yajra/laravel-datatables/issues/2322 and just noticed it today.

Will keep this open for a week incase anyone wants to add their review. Thanks!

yajra commented 1 year ago

Apologies for the late review, released in v10.8.0 🚀 Thanks!

froiden-chetan commented 1 year ago

Our applications are brooked after this update