grav-plugin-admin: Regression: Ajax error in ngnix

Hello, after update to 1.6.3 (and update to 1.3.5) i’ve got issues in the admin pannel : the site seems to work well allthough, exept featherlight wich wouldn’t work before update anyway. As i need a gallery, i also need to deal with this error for i’m sure it has no link between. Thanks!

`Fetch Failed:
can't convert undefined to object

value/<@http://lelogisencorse.ddns.net/user/plugins/admin/themes/grav/js/admin.min.js?1451599962:5:23728
o/<@http://lelogisencorse.ddns.net/user/plugins/admin/themes/grav/js/admin.min.js?1451599962:4:27913

Invalid AJAX response.`

About this issue

Original URL
State: closed
Created 7 years ago
Reactions: 1
Comments: 42 (20 by maintainers)

Commits related to this issue

Fix for nginx and `_url` issue [admin#1244](https://github.com/getgrav/grav-plugin-admin/issues/1244) — committed to getgrav/grav by rhukster 7 years ago

Most upvoted comments

Ok, we’re going to re-release Grav now with this fix.

rhukster on Oct 12, 2017

I’m able to reproduce the issue, trying to locate the issue now. Thanks for the detailed help!

mahagr on Oct 12, 2017

@velsa Please create a new issue from this as nobody looks at closed issues. 😃

They really should though, unless you publish that you follow a workflow where they are deliberately ignored. Is it a notification issue? (let this be a test!)

hughbris on Aug 26, 2020

Ok, i’ve narrowed it down to Nginx, and specifically the try_files part of the configuration. My test site had simply:

location / {
        try_files $uri $uri/ /index.html /index.php;
}

And this worked, however, the webserver-configs/nginx.conf file (and possibly other places has been updated so it looks like this:

location / {
        try_files $uri $uri/ /index.php?_url=$uri&$query_string;
}

When I change it to this, and restart nginx, it breaks with that ajax error. So i removed the &$query_string at the end, and it worked.

So something related to the query_string in this nginx configuration and the changes/fixes we made in the URI class are breaking it… Not fixed yet, but narrowing things down.

rhukster on Oct 12, 2017

@rhukster I can confirm that fixes the issue

mitchellkrogza on Oct 12, 2017