Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

[18:52:41 CDT(-0500)] <dmccallum54> yes

[18:53:42 CDT(-0500)] <JasonElwood> and reasonably easy and quick

[18:53:57 CDT(-0500)] <dmccallum54> you get a 403 in this case i think

[18:56:25 CDT(-0500)] <dmccallum54> would have to either… change all the forms to ajax submits or modify the error page that generates the 403 json to "do something else".

[18:56:41 CDT(-0500)] <dmccallum54> the latter is pretty squishy and isn't really going to be pleasant

[18:57:26 CDT(-0500)] <dmccallum54> for the user i mean

[18:59:44 CDT(-0500)] <dmccallum54> there are jquery plugins for the former (http://www.malsup.com/jquery/form/)

[19:00:14 CDT(-0500)] <dmccallum54> not sure how well that's going to work with our PDF responses

[19:01:45 CDT(-0500)] <dmccallum54> might have to monkey with the headers being set server side

[19:02:16 CDT(-0500)] <dmccallum54> so anyway… maybe an hour or so to see if it's even feasible

[19:03:32 CDT(-0500)] <dmccallum54> "it" being submitting forms via ajax, correctly offering to download resulting PDFs, handling error codes gracefully, and special handling for 403 to offer to redirect to login

[19:09:27 CDT(-0500)] <js70> we don' have a 403 jsp page, I'm pretty sure its bringing back the 500. not a big deal but we need to create the 403 as well.

[19:10:56 CDT(-0500)] <dmccallum54> think it might not be using any of the jsp pages

[19:11:06 CDT(-0500)] <dmccallum54> @PreAuthorize(Permission.PERMIT_ALL)

[19:11:06 CDT(-0500)] <dmccallum54> @ExceptionHandler(AccessDeniedException.class)

[19:11:06 CDT(-0500)] <dmccallum54> @ResponseStatus(HttpStatus.FORBIDDEN)

[19:11:06 CDT(-0500)] <dmccallum54> public @ResponseBody

[19:11:06 CDT(-0500)] <dmccallum54> ServiceResponse handleAccessDenied(final AccessDeniedException e) {

[19:11:06 CDT(-0500)] <dmccallum54> getLogger().error(ERROR_PREFIX, e);

[19:11:06 CDT(-0500)] <dmccallum54> return new ServiceResponse(false, e.getMessage());

[19:11:07 CDT(-0500)] <dmccallum54> }

[19:11:34 CDT(-0500)] <js70> k

[19:12:00 CDT(-0500)] <js70> makes sense

[19:12:48 CDT(-0500)] <dmccallum54> trying to use a page seems a little weird in this case, esp if we're going to try to pretty it up

[19:13:23 CDT(-0500)] <dmccallum54> b/c again you're hitting an API and it's barfing. it's not a web page that's barfing where it would make sense to send you to a nice pretty 'sorry about that' page

[19:13:31 CDT(-0500)] <js70> yeah, I like your jquery form suggestion. if it works should be fairly simple to implement.

[19:18:01 CDT(-0500)] <dmccallum54> isn't that always the story (smile)

[19:18:44 CDT(-0500)] <dmccallum54> alright. i gotta run. daycare pickup calls again.