jQuery Grid Plugin - jqGrid

I realize that the problem we are trying to solve here is because of our authentication mechanism (we are using JAAS), but I'm wondering if anyone has any argument against adding the XMLHttpRequest object to the loadComplete() function call?

Looking at the source code, it's a VERY easy change as the populate function simply defers to jQuery's $.ajax(…) function and the code that calls the jqGrid loadComplete function can simply take the first parameter passed (the request object) to the $.ajax(…).complete(…) function.

The problem is that when JAAS detects that a request is made to a protected resource and the user's session has expired, it redirects the request to the login page.  Agreeably in a somewhat hacky fashion, we add a response header indicating that the user's session is still valid – if the header is missing, we know it's not and can redirect the browser to the login page.

However, because we don't have access to the request object, we can't look for this header.  Obviously, by modifying the grid.base.js file we can “fix” this, but then we have to remember to do this every time we upgrade jqGrid.

So, I'm wondering if anyone has any pushback on making this modification.  Comments are welcome.

–adam