jasig-ssp IRC Logs-2013-03-14

[09:12:17 CDT(-0500)] <JasonElwood> JS- can I screen share to show you what's going on with the reports?

[09:27:56 CDT(-0500)] <js70> sure

[09:28:04 CDT(-0500)] <js70> here is the query for disability

[09:28:07 CDT(-0500)] <js70> select distinct this_.id as y0_ from person this_ inner join person_disability persondisa1_ on this_.person_disability_id=persondisa1_.id where persondisa1_.id is not null and this_.student_type_id is not null

[09:28:19 CDT(-0500)] <js70> pretty simple

[09:29:45 CDT(-0500)] <js70> after that, I just take the id and generate the disabilityreportTO

[09:30:50 CDT(-0500)] <js70> if you get an id then there is something wrong with the projection definition for generating the TO

[09:31:22 CDT(-0500)] <JasonElwood> I emailed the GoTo link

[09:33:07 CDT(-0500)] <js70> got it. voip is acting up. just a sec

[09:34:45 CDT(-0500)] <js70> says 1(213) 493-0014 is a non working number

[09:35:03 CDT(-0500)] <JasonElwood> it looks like the goto meeting number is not working. want to call me? My number is on my email

[09:37:44 CDT(-0500)] <TonyUnicon> i have a dentist's appointment, be back at 10am MT

[10:50:06 CDT(-0500)] <JasonElwood> Dan- I commented on SSP-838. The functionality isn't ideal but it works within the design. My suggestion is to close it with the explanation in the comments

[10:57:36 CDT(-0500)] <dmccallum54> JasonElwood commented on it just now. fine with closing it

[11:09:08 CDT(-0500)] <dmccallum54> ok what's the trick to getting an intake task to show up after submitting the caseload assignment form

[11:11:11 CDT(-0500)] <JasonElwood> great question. I just noticed that it isn't working again this morning

[11:12:03 CDT(-0500)] <JasonElwood> but I was having all kinds of problems with data. most of my accommodation data isn't showing, but I wanted to double-check before reporting

[11:12:17 CDT(-0500)] <dmccallum54> looks like the checkbox value isn't being written into the POST/PUT payload

[11:12:44 CDT(-0500)] <dmccallum54> maybe i horked something with all my git games yesterday

[11:12:51 CDT(-0500)] <dmccallum54> will try backtracking a bit

[11:23:41 CDT(-0500)] <js70> Disability Report is fixed. Taking care of report formatting next.

[11:40:54 CDT(-0500)] <dmccallum54> JasonElwood i'm still poking around, but it looks like the intake checkbox in caseload assignment is only considered if you also set an appointment… is that correct?

[11:41:59 CDT(-0500)] <TonyUnicon> no

[11:42:12 CDT(-0500)] <JasonElwood> it shouldn't be

[11:42:51 CDT(-0500)] <dmccallum54> like i said… still poking around but looks like it's been that way since ~02/26

[11:43:27 CDT(-0500)] <TonyUnicon> thats not the case

[11:43:49 CDT(-0500)] <TonyUnicon> http://onemansblog.com/wp-content/uploads/2009/05/inconceivable.jpg

[11:44:09 CDT(-0500)] <JasonElwood> I only set appt to avoid the required appt constraint by type. I believe I've been doing without setting an appt

[11:44:25 CDT(-0500)] <JasonElwood> idk man. my system is running very strangely right now.

[11:44:35 CDT(-0500)] <TonyUnicon> what is the problem?

[11:45:16 CDT(-0500)] <JasonElwood> having strange/no results saving

[11:46:34 CDT(-0500)] <TonyUnicon> saving caseload assignment?

[11:47:18 CDT(-0500)] <JasonElwood> yes. also self-help guides, accommodations. and self help guides in MyGPS are showing the questions

[11:47:31 CDT(-0500)]

<dmccallum54> fwiw... i only see the intake flag being sent to the server if i set an appt. the JSON for that looks like

Unknown macro: {"id"}

[11:47:55 CDT(-0500)] <dmccallum54> which is consistent w/ the change to CaseloadAssignmentViewController.js here: https://github.com/Jasig/SSP/commit/58ab7eb0b0dc2221c3361bd79f7902a089a28b19

[11:48:17 CDT(-0500)] <dmccallum54> but even then it doesn't seem to result in a task showing up in my user's action plan

[11:48:19 CDT(-0500)] <JasonElwood> In looking at my logs, I'm getting killed with the 500 error

[11:48:28 CDT(-0500)] <dmccallum54> <sigh>

[11:49:38 CDT(-0500)] <JasonElwood> I originally only saw that problem in reports. In the last two days, I've seen it in MyGPS and Accommodations. That's what I've been testing the most lately

[11:50:07 CDT(-0500)] <TonyUnicon> i just did it

[11:57:54 CDT(-0500)] <JasonElwood> I am not able to do it without an appt date and time

[11:58:10 CDT(-0500)] <dmccallum54> yeah…. it's definitely tied to that

[11:58:21 CDT(-0500)] <dmccallum54> am able to get the resulting task to show up for at least one user

[11:58:28 CDT(-0500)] <dmccallum54> not sure why it didnt' work for the other user i tried

[11:58:37 CDT(-0500)] <TonyUnicon> do you mean the required fields on appointment?

[11:59:01 CDT(-0500)] <dmccallum54> well

[11:59:11 CDT(-0500)] <dmccallum54> you can flip those to non-required by selecting the magic student type

[11:59:14 CDT(-0500)] <JasonElwood> could the persistent required fields be causing a problem?

[11:59:15 CDT(-0500)] <TonyUnicon> you can't submit anything on caseload assignment unless those are satisified

[11:59:33 CDT(-0500)] <dmccallum54> flip the type to something non-default then flip it back to EAL

[12:00:36 CDT(-0500)] <JasonElwood> even if the type doesn't require an appt, sometimes the page requires it.

[12:02:23 CDT(-0500)] <TonyUnicon> im still not exactly clear as to what the issue ise

[12:02:35 CDT(-0500)] <JasonElwood> when I successfully created the task, I got this message in the log

[12:02:40 CDT(-0500)] <JasonElwood> DEV: 12:53:06.395 [http-8080-8] INFO t.o.j.s.s.impl.PersonServiceImpl - Read 7 PersonAttributesService coaches and correlated them with 7 local coaches in 30 ms

[12:02:40 CDT(-0500)] <TonyUnicon> if you do not enter appointment date/time

[12:02:40 CDT(-0500)] <JasonElwood> DEV: 12:53:24.918 [http-8080-7] ERROR o.apache.velocity.app.VelocityEngine - ResourceManager : unable to find resource '9d3ce5b1-e27d-40c8-8f45-abcb1bccf3b0-s-1337263200000' in any resource loader.

[12:02:40 CDT(-0500)] <JasonElwood> DEV: 12:53:24.921 [http-8080-7] ERROR o.apache.velocity.app.VelocityEngine - ResourceManager : unable to find resource '9d3ce5b1-e27d-40c8-8f45-abcb1bccf3b0-b-1337263200000' in any resource loader.

[12:02:40 CDT(-0500)] <JasonElwood> DEV: 12:55:03.847 [sspScheduler-3] WARN o.j.s.s.impl.MessageServiceImpl - _ : JavaMailSender was not called; message was marked sent but was not actually sent. To enable mail, update the configuration of the app.

[12:02:40 CDT(-0500)] <JasonElwood> is it related?

[12:02:57 CDT(-0500)] <TonyUnicon> dont think so

[12:03:02 CDT(-0500)] <JasonElwood> appt date and time is not always required. you should be able to send the intake without setting an appt

[12:03:16 CDT(-0500)] <TonyUnicon> yes but what I'm asking is

[12:03:23 CDT(-0500)] <TonyUnicon> is it the case that

[12:03:37 CDT(-0500)] <TonyUnicon> if you don't enter appointment information, the task doesn't get created?

[12:03:54 CDT(-0500)] <dmccallum54> JasonElwood all those log messages are red herrings

[12:04:17 CDT(-0500)] <JasonElwood> yes. either enter an appt date/time or one must exist already.

[12:04:43 CDT(-0500)] <TonyUnicon> Jason your confusing me (smile)

[12:04:48 CDT(-0500)] <TonyUnicon> what is the bug

[12:04:53 CDT(-0500)] <TonyUnicon> lets start there

[12:05:05 CDT(-0500)] <JasonElwood> appt date/time should not be required to send the intake

[12:05:16 CDT(-0500)] <TonyUnicon> so is the bug

[12:05:28 CDT(-0500)] <TonyUnicon> when app date and time is not entered the request does not get sent?

[12:05:36 CDT(-0500)] <JasonElwood> yes

[12:05:46 CDT(-0500)] <TonyUnicon> ok

[12:05:48 CDT(-0500)] <JasonElwood> entered or already exist

[12:06:52 CDT(-0500)] <TonyUnicon> ?

[12:06:57 CDT(-0500)] <TonyUnicon> if its exists already it works

[12:07:13 CDT(-0500)] <JasonElwood> yes

[12:07:49 CDT(-0500)] <JasonElwood> the other issue is that the caseload assignment page sometimes loads with an appointment being required even if the student type doesn't require it. Dan's suggestion is to change the type and back, the requirement goes away.

[12:08:33 CDT(-0500)] <dmccallum54> (there is a old ticket hanging around for that issue)

[12:09:31 CDT(-0500)] <dmccallum54> also… sometimes even when I do specify an appt time and check the intake checkbox i'm not getting a corresponding task created on the backend. i do see the "studentIntakeRequested":true in the json POST, though

[12:10:21 CDT(-0500)] <JasonElwood> right. that issue isn't being addressed now

[12:10:26 CDT(-0500)] <dmccallum54> no

[12:10:27 CDT(-0500)] <dmccallum54> it's not

[12:10:56 CDT(-0500)] <dmccallum54> i think that happens b/c the intake flag handling server side is only triggered by a PUT

[12:10:58 CDT(-0500)] <JasonElwood> just clarifying to further confuse Tony

[12:11:22 CDT(-0500)] <dmccallum54> so perhaps the 1st appointment for a given user will never also set the intake flag

[12:11:42 CDT(-0500)] <dmccallum54> "exciting". that's the word for all this, right?

[12:11:49 CDT(-0500)] <JasonElwood> he may be outside with his head in the snow cooling off

[12:12:14 CDT(-0500)] <JasonElwood> appointment and intake should be completely independent

[12:12:36 CDT(-0500)] <TonyUnicon> ok

[12:12:49 CDT(-0500)] <TonyUnicon> putting the handling on the post as well its not a big deal

[12:15:43 CDT(-0500)] <dmccallum54> if i dial in to the scrum and you hear macklemore & lewis, it's not my fault

[12:17:51 CDT(-0500)] <TonyUnicon> the post code is done

[12:18:00 CDT(-0500)] <TonyUnicon> if there is a ticket for it i'll punt it in

[12:40:47 CDT(-0500)] <JasonElwood> SSP-839

[12:43:54 CDT(-0500)] <TonyUnicon> thanks

[12:46:30 CDT(-0500)] <TonyUnicon> Jason I'm going to edit because it is misleading

[12:46:34 CDT(-0500)] <TonyUnicon> the title

[12:48:43 CDT(-0500)] <JasonElwood> sure

[12:49:28 CDT(-0500)] <JasonElwood> mi título de su título

[12:50:37 CDT(-0500)] <dmccallum54> zomg. back to 404s on the task report. wtf.

[12:50:59 CDT(-0500)] <dmccallum54> i'm sure it's my fault… just sharing the pain.

[12:52:11 CDT(-0500)] <JasonElwood> Serenity Now!

[12:53:42 CDT(-0500)] <dmccallum54> speaking of misleading titles… this 'fastWarDeploy' goal… wow...

[12:55:54 CDT(-0500)] <TonyUnicon> thank god none of this is being logged!

[12:56:02 CDT(-0500)] <TonyUnicon> (smile)

[12:59:48 CDT(-0500)] <TonyUnicon> now I know this is going to be a bad day, in the last three years Ive been coding on a mac eclipse has crashed on exactly once

[13:00:04 CDT(-0500)] <TonyUnicon> until today

[13:18:49 CDT(-0500)] <js70> Jason if you would run this query and give the the results, that would be awesome:

[13:19:05 CDT(-0500)] <js70> select distinct journalent5_.id as y0_, person1_.first_name as y1_, person1_.middle_name as y2_, person1_.last_name as y3_, person1_.school_id as y4_, person1_.primary_email_address as y5_, person1_.secondary_email_address as y6_, person1_.cell_phone as y7_, person1_.home_phone as y8_, person1_.address_line_1 as y9_, person1_.address_line_2 as y10_, person1_.city as y11_, person1_.state as y12_, person1_.zip_code as y13_, person1_.id as y14

[13:22:23 CDT(-0500)] <JasonElwood> can you re-send the fully query

[13:22:47 CDT(-0500)] <dmccallum54> gist.github.com ftw

[13:39:20 CDT(-0500)] <js70> here it is: https://gist.github.com/jstanley70/09aac9ad61c511cf55be

[13:40:29 CDT(-0500)] <JasonElwood> no results

[13:42:11 CDT(-0500)] <js70> did you change the ids to match your journal_step_detail?

[13:42:32 CDT(-0500)] <js70> ids?

[13:44:38 CDT(-0500)] <JasonElwood> hold on. wrong id

[13:47:35 CDT(-0500)] <JasonElwood> dupes

[13:48:18 CDT(-0500)] <js70> can you send me the results so I can look at what might be causing the dupes

[13:51:07 CDT(-0500)] <JasonElwood> gisted

[13:51:14 CDT(-0500)] <js70> url?

[13:51:29 CDT(-0500)] <JasonElwood> commented on your's

[13:51:35 CDT(-0500)] <js70> got it thans.

[13:54:08 CDT(-0500)] <TonyUnicon1> initportal zzzzz

[13:54:23 CDT(-0500)] <dmccallum54> no such target!

[13:54:29 CDT(-0500)] <TonyUnicon1> lol

[13:55:03 CDT(-0500)] <dmccallum54> so here's a cool one

[13:55:53 CDT(-0500)] <dmccallum54> log in to uP… go to MyGPS… see my task list… log out… nav directly to the MyGPS splash URL… i see almost all tasks in the db… for everyone… but of course not all the tasks in the db… that would be ridiculous

[13:56:16 CDT(-0500)] <dmccallum54> i'm wondering if my db just got horked by the whole singleton-anonymous-user business

[13:56:25 CDT(-0500)] <dmccallum54> so wiping out db and starting over

[13:59:29 CDT(-0500)] <TonyUnicon1> heh even better

[13:59:50 CDT(-0500)] <TonyUnicon1> on chrome I see lots of wierd tasks

[13:59:56 CDT(-0500)] <TonyUnicon1> on firefox I see none!

[14:00:18 CDT(-0500)] <dmccallum54> i'm going to kill myself if we end up recommending IE

[14:02:24 CDT(-0500)] <dmccallum54> oh wait, that'd never work. this project being some sort of record-setter for the "dangling comma" javascript syntax violation

[14:03:08 CDT(-0500)] <js70> I know the problem…but I'm not telling (to Journal report)

[14:03:17 CDT(-0500)] <js70> Should have a fix soon.

[14:04:28 CDT(-0500)] <TonyUnicon1> heh Dan

[14:04:42 CDT(-0500)] <TonyUnicon1> so what happens is

[14:04:59 CDT(-0500)] <TonyUnicon1> it will grab tasks for the session

[14:05:08 CDT(-0500)] <TonyUnicon1> regardless of user

[14:05:39 CDT(-0500)] <TonyUnicon1> and it records the session if you are authenticated

[14:06:09 CDT(-0500)] <TonyUnicon1> in otherwords

[14:06:18 CDT(-0500)] <TonyUnicon1> it does a select * from task where sessionId = ?

[14:06:21 CDT(-0500)] <TonyUnicon1> what it should do

[14:07:01 CDT(-0500)] <TonyUnicon1> is select * from task where session = ? and person_id = (either null or anonymousId not sure yet)

[14:07:45 CDT(-0500)] <TonyUnicon1> so that bug was there

[14:07:55 CDT(-0500)] <TonyUnicon1> fixing the anonymous user issue only exposed it

[14:08:46 CDT(-0500)] <TonyUnicon1> you can test this by opening another browser

[14:08:54 CDT(-0500)] <TonyUnicon1> go in unauthenticated

[14:09:03 CDT(-0500)] <TonyUnicon1> and you should see nothing or at least different tasks

[14:09:23 CDT(-0500)] <dmccallum54> yeah. an incognito window seemed to work fine

[14:10:30 CDT(-0500)] <TonyUnicon1> so it looks like it has to be select * from task where session = ? and person_id = anonymousId

[14:10:39 CDT(-0500)] <TonyUnicon1> Ill write up a ticket for it

[14:10:43 CDT(-0500)] <dmccallum54> thx

[14:11:28 CDT(-0500)] <js70> jason: for purposes of the report can we require a current program status ie expiration_date must be null. What's happening is your student has had a previous coach with an expired program status. therefore we are getting 2 rows one for each coach.

[14:17:35 CDT(-0500)] <JasonElwood> Those students should have a current program status

[14:20:18 CDT(-0500)] <dmccallum54> so what is an anonymous user supposed to do to wipe out a task list?

[14:20:39 CDT(-0500)] <dmccallum54> there's no way via the UI to switch to a different anonymous session

[14:23:09 CDT(-0500)] <dmccallum54> think they're basically stuck with something awful like "please clear your cache and delete cookies"

[14:24:03 CDT(-0500)] <js70> Jason, they do have a current status, they also have an expired status. I just want to confirm 2 things.

[14:24:20 CDT(-0500)] <js70> 1. that we are only interested in current status

[14:24:49 CDT(-0500)] <TonyUnicon1> Dan, yes

[14:25:00 CDT(-0500)] <js70> 2. that every student must have a current status to appear in reports or they can have a current status or a null status.

[14:25:49 CDT(-0500)] <js70> the issue is the student has both a current status and an expired status which generates 2 rows for each entry.

[14:26:14 CDT(-0500)] <js70> we can't use the person entity because the reports take too long so we get this duplication issue.

[14:26:30 CDT(-0500)] <dmccallum54> JasonElwood that seem like a big problem to you? not being able to reset an anonymous session?

[14:27:58 CDT(-0500)] <js70> Jason also can the student have only one current program status at any given time.

[14:30:51 CDT(-0500)] <TonyUnicon1> We could allow the student to delete tasks

[14:31:05 CDT(-0500)] <TonyUnicon1> which we do

[14:31:12 CDT(-0500)] <TonyUnicon1> but unless if its flagged as deletable

[14:31:16 CDT(-0500)] <JasonElwood> JS- there are two active statuses, one with a date on null.

[14:31:32 CDT(-0500)] <JasonElwood> So are you suggesting that we just exclude expired program_Statuses from the results?

[14:32:58 CDT(-0500)] <JasonElwood> technically the student could have more than one status but the tool shouldn't create that situation

[14:33:18 CDT(-0500)] <JasonElwood> so theoretically, there would only be on valid program status

[14:33:22 CDT(-0500)] <TonyUnicon1> task session fix is in

[14:33:44 CDT(-0500)] <js70> or possibliy a null program status?

[14:34:38 CDT(-0500)] <JasonElwood> so the anonymous session is saved on the server and would retain tasks added by somebody else?

[14:35:12 CDT(-0500)] <JasonElwood> meaning a single user with diff browsers or concurrent users would step on each other

[14:35:21 CDT(-0500)] <dmccallum54> well

[14:35:29 CDT(-0500)] <dmccallum54> i believe tony has or is fixing that issue

[14:35:33 CDT(-0500)] <js70> So should I enforce this rule in the reports (all the reports), only enforce it in this report or compress the rows?

[14:35:36 CDT(-0500)] <dmccallum54> what i'm talking about is more for a single browser

[14:36:19 CDT(-0500)] <dmccallum54> e.g. you're an anon user, you set up some tasks, print 'em and walk away

[14:36:55 CDT(-0500)] <JasonElwood> without the session closing, the tasks remain

[14:37:00 CDT(-0500)] <dmccallum54> right

[14:37:20 CDT(-0500)] <dmccallum54> so they probably eventually disappear if the anonymous session times out

[14:37:39 CDT(-0500)] <dmccallum54> or you do something to wipe out your session cookie

[14:37:41 CDT(-0500)] <TonyUnicon1> yes Dan that issue is now checked in

[14:38:46 CDT(-0500)] <JasonElwood> is it too late to re-write MyGPS?

[14:39:26 CDT(-0500)] <dmccallum54> well

[14:39:37 CDT(-0500)] <dmccallum54> Tony is right that anonymous user handling in general needs a hard look

[14:39:43 CDT(-0500)] <JasonElwood> I just hear Tony scream yes

[14:39:49 CDT(-0500)] <dmccallum54> but that shit's all over the place

[14:40:06 CDT(-0500)] <TonyUnicon1> ^^

[14:40:08 CDT(-0500)] <TonyUnicon1> yes

[14:40:36 CDT(-0500)] <TonyUnicon1> lets not talk about if someone actually creates a uportal user named anonymousUser

[14:40:36 CDT(-0500)] <JasonElwood> I don't know what Tony's solution is, but is there enough risk that anonymous should go away for now?

[14:40:43 CDT(-0500)] <TonyUnicon1> we would likely create a singularity

[14:41:09 CDT(-0500)] <TonyUnicon1> there is more risk in trying to remove it at this stage of the release

[14:41:15 CDT(-0500)] <JasonElwood> ok

[14:41:16 CDT(-0500)] <TonyUnicon1> but it should be addressed

[14:41:39 CDT(-0500)] <TonyUnicon1> OR

[14:41:48 CDT(-0500)] <TonyUnicon1> even the id of the anonymous user somehow gets change

[14:41:56 CDT(-0500)] <TonyUnicon1> then the whole thing crumbles

[14:42:17 CDT(-0500)] <dmccallum54> yeah

[14:42:25 CDT(-0500)] <JasonElwood> that sounds like substantial risk

[14:43:14 CDT(-0500)] <TonyUnicon1> but its a risk we take elsewhere

[14:43:36 CDT(-0500)] <TonyUnicon1> the whole dependency on technical id's is just a bad idea IMO

[14:43:44 CDT(-0500)] <dmccallum54> i think the right approach is a basically a per-session lightly authenticated anonymous user. i.e. each anonymous user is really a separate user record bound to a potentially very long-lived quasi-session

[14:43:52 CDT(-0500)] <dmccallum54> bascially the same thing amazon does

[14:44:25 CDT(-0500)] <JasonElwood> JS- I'm not seeing the duplicates in other reports right now. I know there was one before that was fixed. Since it's only happening here, let's enforce it in this report.

[14:44:42 CDT(-0500)] <dmccallum54> so an end user always has a distinct identity and standard ways of assuming and abandoning that identity

[14:44:55 CDT(-0500)] <js70> perfect

[14:45:25 CDT(-0500)] <JasonElwood> Dan & Tony- should we Hangout for this issue?

[14:45:33 CDT(-0500)] <JasonElwood> or do you have enough handle on it?

[14:45:37 CDT(-0500)] <dmccallum54> well

[14:45:47 CDT(-0500)] <dmccallum54> for 1.2.0 we can't undertake a major refactor

[14:45:51 CDT(-0500)] <dmccallum54> unless that date slips

[14:45:58 CDT(-0500)] <dmccallum54> like, alot

[14:46:03 CDT(-0500)] <TonyUnicon1> yeah I think its a really bad idea to change this mechanic now

[14:46:22 CDT(-0500)] <dmccallum54> so i think we're just in whack-a-mole mode

[14:46:31 CDT(-0500)] <TonyUnicon1> yeah

[14:46:49 CDT(-0500)] <JasonElwood> if this becomes a real problem for schools, they could always setup guides to require authentication

[14:47:02 CDT(-0500)] <dmccallum54> and we also need to decide what, if anything is to be done about the lack of a feature for user-initiated abandonment of an anonymous session

[14:47:07 CDT(-0500)] <JasonElwood> I believe that would only leave the search exposed to adding tasks

[14:47:30 CDT(-0500)] <dmccallum54> i wonder how often anonymous mode is actually used in the wild

[14:48:10 CDT(-0500)] <TonyUnicon1> if it does work properly at sinclair, like printing does

[14:48:24 CDT(-0500)] <TonyUnicon1> it really does make me wonder if the source that was ported

[14:48:26 CDT(-0500)] <TonyUnicon1> as the latest

[14:48:38 CDT(-0500)] <TonyUnicon1> was the latest*

[14:48:39 CDT(-0500)] <JasonElwood> according to Sinclair, a lot. they estimate 40% of students get service without an id because they walk in and get all services in one day before their id is created

[14:49:01 CDT(-0500)] <JasonElwood> we confirmed that printing does work last night

[14:49:26 CDT(-0500)] <TonyUnicon1> yeah so why did it not work for us?

[14:49:32 CDT(-0500)] <TonyUnicon1> same thing with task images

[14:49:37 CDT(-0500)] <TonyUnicon1> that I had to fix

[14:49:43 CDT(-0500)] <TonyUnicon1> they were logical errors

[14:49:51 CDT(-0500)] <TonyUnicon1> that obviously worked at one time

[14:50:09 CDT(-0500)] <dmccallum54> i thought this code isn't what sinclair is running at all

[14:50:25 CDT(-0500)] <dmccallum54> that it's just supposed to do the same stuff but is otherwise a rewrite

[14:50:28 CDT(-0500)] <JasonElwood> they are running it, except for the student intake

[14:50:29 CDT(-0500)] <TonyUnicon1> their client definitely is the same or close

[14:50:33 CDT(-0500)] <dmccallum54> k

[14:50:46 CDT(-0500)] <dmccallum54> the back-end is all new, tho?

[14:51:02 CDT(-0500)] <TonyUnicon1> i dont know the history, I had asked this question before in a standup a few weeks ago

[14:51:20 CDT(-0500)] <JasonElwood> right

[14:52:26 CDT(-0500)] <TonyUnicon1> i can understand if the issues were around shifting resource directories and such

[14:52:31 CDT(-0500)] <TonyUnicon1> stuff that can happen in a code migration

[14:52:41 CDT(-0500)] <TonyUnicon1> but logical errors that suggests the feature never worked

[14:52:48 CDT(-0500)] <TonyUnicon1> is strange when screenshots say otherwise

[14:53:48 CDT(-0500)] <dmccallum54> k. so

[14:53:50 CDT(-0500)] <TonyUnicon1> well it keeps things exciting at the very least

[14:54:16 CDT(-0500)] <JasonElwood> so what question is on the table?

[14:54:19 CDT(-0500)] <dmccallum54> i think for known GPS task issues we're down to SSP-836 which is on me to see if i can even reproduce again

[14:54:43 CDT(-0500)] <dmccallum54> …then also a decision on whether or not anything is to be done about anonymous session reset

[14:54:43 CDT(-0500)] <TonyUnicon1> not a question, just a doubt Jason

[14:54:55 CDT(-0500)] <TonyUnicon1> that the code that was migrated was the latest and greatest

[14:57:31 CDT(-0500)] <dmccallum54> JasonElwood i'm going to grab some food. i guess noodle on the anon session reset thing… a little concerned it's going to take a while to get working if we want to do something there if we end up needing to do it after Tony is out of pocket

[14:58:04 CDT(-0500)] <dmccallum54> assuming we decide to not do anything there, i'll poke around on 836 a little bit when i get back

[14:58:09 CDT(-0500)] <JasonElwood> so the options are what? leave it and put in a message about closing the browser OR ???

[15:00:21 CDT(-0500)] <dmccallum54> put in a message about closing the browser or add a quasi logout/reset button that'll have the effect of nuking cookies

[15:00:47 CDT(-0500)] <dmccallum54> fwiw, just closing the browser and reopening it does not work

[15:02:06 CDT(-0500)] <dmccallum54> well… i should take that back… i need to deploy tony's latest code before i assert that

[15:02:15 CDT(-0500)] <JasonElwood> could also nuke the cookies when it is printed/emailed

[15:02:24 CDT(-0500)] <TonyUnicon1> that should not work

[15:02:33 CDT(-0500)] <TonyUnicon1> @Dan

[15:03:30 CDT(-0500)] <dmccallum54> yes?

[15:05:01 CDT(-0500)] <TonyUnicon1> @Dan that should not work (open/closing browser resets tasks)

[15:08:43 CDT(-0500)] <JasonElwood> so maybe all three: msg about closing browser, simple reset button and reset when print/email

[15:14:36 CDT(-0500)] <dmccallum54> TonyUnicon1 doesn't work b/c chrome doesn't kill session cookies?

[15:14:57 CDT(-0500)] <TonyUnicon1> yeah

[15:15:40 CDT(-0500)] <dmccallum54> yeah so jason i think if anything is to be done it would need to be a button

[15:15:44 CDT(-0500)] <TonyUnicon1> its dependent on the cookie, not the opening/closing of browsers, so if a browser does remove the cooking on browser close then yeah opening/closing would clean it

[15:16:12 CDT(-0500)] <dmccallum54> yeah, chrome doesn't wipe session cookies. FF used to, but i think that's changed recently

[15:16:23 CDT(-0500)] <dmccallum54> i dont like reset on print/email

[15:16:40 CDT(-0500)] <dmccallum54> i cant think of another app where pressing print also has a destructive side-effect

[15:17:35 CDT(-0500)] <JasonElwood> this one does. you can't go back into the intake after you save it

[15:17:53 CDT(-0500)] <TonyUnicon1> thats not destructive

[15:17:56 CDT(-0500)] <JasonElwood> half of what Tony has worked on in the last week has been workarounds for the existing flaws

[15:18:00 CDT(-0500)] <TonyUnicon1> just exclusive (smile)

[15:18:55 CDT(-0500)] <JasonElwood> I'm not saying I love it, but it does put the action within the use case. whereas closing the browser or a reset button is an extra step

[15:19:21 CDT(-0500)] <TonyUnicon1> I have to agree with Dan

[15:19:28 CDT(-0500)] <TonyUnicon1> that is really unintuitive

[15:19:55 CDT(-0500)] <TonyUnicon1> so the only side effect of long lived sessions

[15:19:57 CDT(-0500)] <JasonElwood> the best option is a reset button then

[15:19:57 CDT(-0500)] <TonyUnicon1> is Task noise

[15:20:02 CDT(-0500)] <TonyUnicon1> right?

[15:20:14 CDT(-0500)] <TonyUnicon1> So why not let all Tasks be deletable always

[15:20:32 CDT(-0500)] <JasonElwood> all or all anonymous?

[15:20:41 CDT(-0500)] <TonyUnicon1> anonymous tasks

[15:21:19 CDT(-0500)] <JasonElwood> the issue will be that anonymous users are really concerned about the session because it doesn't contain any personal info. walking away as is doesn't harm them

[15:22:04 CDT(-0500)] <TonyUnicon1> right?

[15:22:09 CDT(-0500)] <dmccallum54> yes (if i understand correctly)… this is a usability, not a security issue

[15:22:17 CDT(-0500)] <TonyUnicon1> yes

[15:22:40 CDT(-0500)] <TonyUnicon1> making anonymous tasks deletable is a lot less intrusive

[15:22:43 CDT(-0500)] <TonyUnicon1> in the eleventh hour

[15:22:46 CDT(-0500)] <JasonElwood> users aren't going to spend the time deleting all the tasks. especially if they could hit a single reset button

[15:23:04 CDT(-0500)] <JasonElwood> and it's again an extra step people won't do

[15:23:12 CDT(-0500)] <TonyUnicon1> but we dont know the side effects of that

[15:23:29 CDT(-0500)] <dmccallum54> side-effects of a reset button?

[15:23:33 CDT(-0500)] <TonyUnicon1> yes

[15:23:52 CDT(-0500)] <JasonElwood> I understand reset as wipe out the session and the tasks that are saved

[15:23:53 CDT(-0500)] <TonyUnicon1> spring security may freak and boot the user from the session

[15:24:19 CDT(-0500)] <JasonElwood> isn't that what we want?

[15:24:26 CDT(-0500)] <TonyUnicon1> when they print?

[15:25:31 CDT(-0500)] <JasonElwood> when they are done

[15:25:55 CDT(-0500)] <TonyUnicon1> what does done mean though?

[15:26:11 CDT(-0500)] <JasonElwood> our issue here is that tasks remain for anonymous session

[15:26:14 CDT(-0500)] <JasonElwood> right?

[15:26:19 CDT(-0500)] <TonyUnicon1> yeah

[15:26:32 CDT(-0500)] <TonyUnicon1> if the cookie is alive

[15:31:02 CDT(-0500)] <dmccallum54> the 'done' i've been talking about would be up to the end user

[15:31:14 CDT(-0500)] <dmccallum54> i.e. they push a button for that purpose

[15:31:36 CDT(-0500)] <dmccallum54> and it either a) deletes their current domain cookie or b) calls a logout function on the server to the same effect

[15:31:57 CDT(-0500)] <dmccallum54> not sure we need to do any actual server-side cleanup in the db

[15:32:00 CDT(-0500)] <JasonElwood> I agree. For somebody taking the extra step, pushing a single button covers what we want. If they don't, we have no control.

[15:32:16 CDT(-0500)] <dmccallum54> if we wanted "b" it would be just to conserve session-related resources (memory)

[15:35:07 CDT(-0500)] <JasonElwood> I'm ok with either way. the button covers the most of what we want

[15:44:07 CDT(-0500)] <JasonElwood> do we have an agreement and task?

[15:44:27 CDT(-0500)] <dmccallum54> not yet i dont think

[15:44:47 CDT(-0500)] <dmccallum54> so it's not w/o risk to add the done button… what else is outstanding that would be considered higher priority?

[15:46:11 CDT(-0500)] <JasonElwood> tony has 836 and 521

[15:46:21 CDT(-0500)] <dmccallum54> give 836 to me

[15:46:53 CDT(-0500)] <JasonElwood> 521 is the username creation

[15:47:06 CDT(-0500)] <dmccallum54> how far is 521 from done?

[15:47:53 CDT(-0500)] <dmccallum54> JasonElwood i think a 'done' button is going to end up bleeding into next week

[15:48:34 CDT(-0500)] <TonyUnicon1> 521?

[15:48:39 CDT(-0500)] <dmccallum54> putting words into tony's mouth… it sounds like a more wide-open task delete option is more doable if the goal is having something done this week

[15:48:50 CDT(-0500)] <dmccallum54> TonyUnicon1 521 yes

[15:48:53 CDT(-0500)] <TonyUnicon1> SSp-821 was the triage

[15:49:02 CDT(-0500)] <TonyUnicon1> to 521 which we said we wouldnt do

[15:49:10 CDT(-0500)] <TonyUnicon1> its in the comments

[15:50:32 CDT(-0500)] <JasonElwood> 521- wouldn't do or not do now?

[15:50:34 CDT(-0500)] <dmccallum54> ok so there's still gaps where you might not have pressed 'retrieve from external'

[15:50:39 CDT(-0500)] <dmccallum54> ?

[15:50:49 CDT(-0500)] <TonyUnicon1> depends on what you mean by gap

[15:50:53 CDT(-0500)] <TonyUnicon1> we now have control over the username

[15:51:01 CDT(-0500)] <TonyUnicon1> which is what made 531 so painful

[15:51:06 CDT(-0500)] <TonyUnicon1> 521*

[15:51:18 CDT(-0500)] <TonyUnicon1> but we dont force users to sync

[15:51:54 CDT(-0500)] <dmccallum54> end result being i can create a person with the "wrong" username?

[15:52:16 CDT(-0500)] <dmccallum54> but if the schoolId is correct and background sync is enabled, it would eventually be corrected?

[15:52:43 CDT(-0500)] <TonyUnicon1> i dont know I think you know more about it than I do

[15:52:48 CDT(-0500)] <TonyUnicon1> but they certainly can correct it

[15:52:51 CDT(-0500)] <TonyUnicon1> if there is usererror

[15:53:16 CDT(-0500)] <JasonElwood> I thought edit was only available if sync is turned off

[15:53:27 CDT(-0500)] <dmccallum54> caseload assignment is special

[15:53:39 CDT(-0500)] <dmccallum54> there's nothing that forces you to retrieve from external

[15:54:01 CDT(-0500)] <dmccallum54> and fields aren't locked down until you do that

[15:54:27 CDT(-0500)] <TonyUnicon1> right

[15:55:03 CDT(-0500)] <TonyUnicon1> the whole reason we exposed the field was we weren't ready to take on 521

[15:55:05 CDT(-0500)] <dmccallum54> 821 doesn't address that but it does give you a fighting chance at getting the username field correct if you're creating a new user

[15:55:49 CDT(-0500)] <dmccallum54> you're right tony… from 521: "If external integration is enabled, I wonder if you shouldn't be allowed to submit the form at all if you've specified a schoolId, but not successfully retrieved a corresponding external record."

[15:56:16 CDT(-0500)] <dmccallum54> formulated as a question, but that's the distinction btwn 521 and 821

[15:56:29 CDT(-0500)] <TonyUnicon1> si

[15:57:17 CDT(-0500)] <dmccallum54> are we to the point of needing a call?

[15:57:24 CDT(-0500)] <dmccallum54> the scrum bridge is always available

[15:58:21 CDT(-0500)] <JasonElwood> Tony- did any of the recent changes you made involve the o.j.ssp.web.api.SessionController

[15:59:01 CDT(-0500)] <dmccallum54> imho… if the two candidate issues for tony right now are 521 and the 'done' button i'd rather he spend a little bit of time on 688 b/c it continues to be a mystery and is truly crippling

[15:59:06 CDT(-0500)] <TonyUnicon1> off hand I don't know I can look it off easy enough

[15:59:42 CDT(-0500)] <TonyUnicon1> look it up*

[15:59:55 CDT(-0500)] <JasonElwood> I'm testing the intake in MyGPS now and get a proxy h andle is no longer valid error

[16:00:24 CDT(-0500)] <TonyUnicon1> SessionController has been touched but not by me

[16:00:38 CDT(-0500)] <TonyUnicon1> why?

[16:00:52 CDT(-0500)] <dmccallum54> think the last commit on SessionController was me back in Jan (656f2faab448f64e90b68a5f03b5d9b4652d3bd8)

[16:01:21 CDT(-0500)] <JasonElwood> ERROR o.j.ssp.web.api.SessionController - Error:

[16:01:21 CDT(-0500)] <JasonElwood> org.hibernate.HibernateException: proxy handle is no longer valid

[16:01:50 CDT(-0500)] <TonyUnicon1> hmm

[16:02:15 CDT(-0500)] <TonyUnicon1> proxies in hibernate involve collections

[16:02:29 CDT(-0500)] <TonyUnicon1> not likely related to anything in sessioncontroller

[16:02:31 CDT(-0500)] <TonyUnicon1> what are you doing?

[16:03:25 CDT(-0500)] <dmccallum54> alright. i'm getting food. for reals.

[16:03:25 CDT(-0500)] <TonyUnicon1> an error like that comes from things not being attached properly to a hibernate session I think

[16:03:57 CDT(-0500)] <JasonElwood> so here's what happened. successfully created an intake request. logged in as a student. completed the intake, clicked save. got that error. clicked ok. now I have an unauthenticated MyGPS home page (login button, no contact coach) but the students tasks are in the list

[16:05:43 CDT(-0500)] <TonyUnicon1> can you pull the full stack from the log?

[16:05:52 CDT(-0500)] <TonyUnicon1> the student tasks are there because your code is not up to date

[16:06:16 CDT(-0500)] <JasonElwood> it is up to date

[16:06:22 CDT(-0500)] <JasonElwood> with the repo

[16:06:29 CDT(-0500)] <TonyUnicon1> as of when

[16:06:42 CDT(-0500)] <TonyUnicon1> in anycase

[16:06:44 CDT(-0500)] <TonyUnicon1> the stack would be great

[16:07:00 CDT(-0500)] <JasonElwood> 0 commits behind

[16:07:56 CDT(-0500)] <JasonElwood> clicked on a task and the page refreshed with logged in appearance

[16:13:18 CDT(-0500)] <TonyUnicon1> so I don't think this is a MyGPS bug

[16:13:29 CDT(-0500)] <TonyUnicon1> this is either a hibernate bug or an ssp bug

[16:13:36 CDT(-0500)] <TonyUnicon1> doing a little more research

[16:14:52 CDT(-0500)] <JasonElwood> idk man. just tried again and it worked. 2 fails, 1 success

[16:15:02 CDT(-0500)] <TonyUnicon1> different students?

[16:15:09 CDT(-0500)] <JasonElwood> nope

[16:15:18 CDT(-0500)] <TonyUnicon1> did you change their special service group?

[16:15:24 CDT(-0500)] <JasonElwood> nope

[16:16:56 CDT(-0500)] <TonyUnicon1> it says it can be caused by a transaction timeout

[16:19:26 CDT(-0500)] <TonyUnicon1> the entry point for that stack is

[16:19:26 CDT(-0500)] <TonyUnicon1> at org.jasig.ssp.web.api.SessionController.getAuthenticatedPerson(SessionController.java:137) ~[SessionController.class:na]

[16:20:02 CDT(-0500)] <TonyUnicon1> it doesn't go through any mygps code

[16:20:17 CDT(-0500)] <TonyUnicon1> but I don't know why its happening

[16:20:17 CDT(-0500)] <JasonElwood> cool. thanks.

[16:20:29 CDT(-0500)] <JasonElwood> another one of the 500 type errors

[16:20:52 CDT(-0500)] <TonyUnicon1> yeah its a strange one

[16:20:59 CDT(-0500)] <TonyUnicon1> it could be a canary though

[16:21:22 CDT(-0500)] <TonyUnicon1> what is happening is

[16:21:29 CDT(-0500)] <TonyUnicon1> when we load the person

[16:21:38 CDT(-0500)] <TonyUnicon1> certain children objects are lazy loaded

[16:22:05 CDT(-0500)] <TonyUnicon1> and what hibernate does, is it won't fetch a collection until someone calls it's getter

[16:22:28 CDT(-0500)] <TonyUnicon1> and it does this through a proxy

[16:22:39 CDT(-0500)] <TonyUnicon1> but it the object is lost to the hibernate session

[16:22:42 CDT(-0500)] <TonyUnicon1> you get this error

[16:22:52 CDT(-0500)] <TonyUnicon1> but this should not happen

[16:23:03 CDT(-0500)] <TonyUnicon1> because all controller methods are wrapped in transactional boundaries managed by spring

[16:23:24 CDT(-0500)] <JasonElwood> if we are sure it doesn't have anything to do with the session changes in MyGPS, I can chalk it up to a fluke until it happens again

[16:23:39 CDT(-0500)] <TonyUnicon1> i would get Dan's opinion on it

[16:24:05 CDT(-0500)] <TonyUnicon1> what I'm saying is this is different than the other 500

[16:24:10 CDT(-0500)] <TonyUnicon1> could be a signal of a greater problem

[16:24:13 CDT(-0500)] <JasonElwood> I know

[16:24:33 CDT(-0500)] <TonyUnicon1> this is not from saving the intake form

[16:24:38 CDT(-0500)] <TonyUnicon1> its from fetching the authenticated user

[16:24:41 CDT(-0500)] <TonyUnicon1> which is common code

[16:25:17 CDT(-0500)] <TonyUnicon1> probably not what you want to hear

[16:25:59 CDT(-0500)] <JasonElwood> at this point, it is what it is. I haven't been able to finish a single test today

[17:12:56 CDT(-0500)] <JasonElwood> My repo is up to date with rel-1-2, but I'm still seeing that the intake request doesn't create a task unless there is an appointment

[17:13:20 CDT(-0500)] <TonyUnicon1> sweet

[17:13:38 CDT(-0500)] <JasonElwood> I activated a new student to work with

[17:14:10 CDT(-0500)] <TonyUnicon1> did you request the intake when you first entered him in?

[17:14:18 CDT(-0500)] <TonyUnicon1> or after?

[17:14:48 CDT(-0500)] <JasonElwood> first entered without an appt didn't work

[17:15:03 CDT(-0500)] <JasonElwood> then went back and added the appt and request. that worked

[17:15:46 CDT(-0500)] <TonyUnicon1> reopen it and send it back to me

[17:15:53 CDT(-0500)] <dmccallum54> TonyUnicon1 think it's b/c of line 466 in CaseloadAssignmentViewController.js

[17:16:29 CDT(-0500)] <TonyUnicon1> ok

[17:16:32 CDT(-0500)] <TonyUnicon1> thank you

[17:16:36 CDT(-0500)] <TonyUnicon1> probably saved me 30 minutes

[17:17:01 CDT(-0500)] <TonyUnicon1> AbstractSessionViewModel.js

[17:17:19 CDT(-0500)] <TonyUnicon1> is where all view models inherit from

[17:17:30 CDT(-0500)] <TonyUnicon1> sorry, ment to be pvt

[17:19:42 CDT(-0500)] <JasonElwood> please clarify on 521. postpone or close?

[17:19:54 CDT(-0500)] <TonyUnicon1> postpone

[17:20:00 CDT(-0500)] <JasonElwood> thanks

[17:58:40 CDT(-0500)] <dmccallum54> JasonElwood shot in the dark on 688… any chance you don't have the latest platform source code?

[17:59:29 CDT(-0500)] <JasonElwood> looks like I'm 1 behind rel-1-1-patches

[17:59:59 CDT(-0500)] <dmccallum54> hmmm

[18:00:01 CDT(-0500)] <dmccallum54> that's not enough

[18:00:07 CDT(-0500)] <dmccallum54> what's your uportal-war/src/main/java/org/jasig/portal/rest/SessionRESTController.java look like

[18:02:19 CDT(-0500)] <TonyUnicon1> one minute, let finish this checkin

[18:07:45 CDT(-0500)] <GregElliott> Hi, JasonElwood.

[18:07:50 CDT(-0500)] <JasonElwood> Am i the only one getting this error regularly?

[18:07:59 CDT(-0500)] <JasonElwood> what's up Greg

[18:08:09 CDT(-0500)] <GregElliott> ah, I see Dan was already asking.

[18:08:17 CDT(-0500)] <GregElliott> about 1 of the files in question.

[18:08:23 CDT(-0500)] <dmccallum54> i was several weeks ago until i freshened up my platform

[18:08:39 CDT(-0500)] <dmccallum54> jim has been seeing it until i guess maybe today? greg has more info...

[18:09:10 CDT(-0500)] <GregElliott> I was able to consistently get the 500 if I waited for the KSA interval to expire, but before the uP session to expire (what would be prevailing in the normal build.)

[18:09:33 CDT(-0500)] <TonyUnicon1> https://gist.github.com/aarland/211cbe33f68f7f5c3086

[18:09:43 CDT(-0500)] <GregElliott> the onther condition was having SessionRESTController.java not have the .json extension in the mapping.

[18:11:14 CDT(-0500)] <GregElliott> when made to match with .json on end, when reached the uP timeout, would get access denied and dialog prompt to login again.

[18:12:57 CDT(-0500)] <GregElliott> Tony, that looks correct.

[18:13:16 CDT(-0500)] <dmccallum54> GregElliot when KSA was triggered but the uP session hadn't expired you would get the 688 stack trace?

[18:13:18 CDT(-0500)] <TonyUnicon1> what do I win?

[18:13:27 CDT(-0500)] <GregElliott> yes.

[18:13:39 CDT(-0500)] <GregElliott> yes, dmccallum54

[18:13:51 CDT(-0500)] <dmccallum54> was that with or without the UP-3661 patch?

[18:14:46 CDT(-0500)] <GregElliott> that was without (ie, KSA had session.json, SessionRESTController had session)

[18:15:18 CDT(-0500)] <dmccallum54> k. so you've still yet to see 688 as long as you're running with UP-3661, no matter what you do to wiggle timeouts up and down?

[18:15:28 CDT(-0500)] <GregElliott> correct.

[18:15:51 CDT(-0500)] <dmccallum54> can you clarify what jim's been seeing today?

[18:16:30 CDT(-0500)] <GregElliott> Jim saw the error early morning. he said he'd run a report, then run it again, and on the subsequent run, he'd see the error.

[18:16:53 CDT(-0500)] <GregElliott> perhaps he's going past the uP session timeout of 30 min

[18:16:55 CDT(-0500)] <dmccallum54> but then he stopped seeing the 688 error at some point?

[18:17:02 CDT(-0500)] <GregElliott> but he hadn't seen it for the rest of today.

[18:17:15 CDT(-0500)] <dmccallum54> did he update platform in some way?

[18:17:21 CDT(-0500)] <GregElliott> and he did say he did a fresh pull fromplatform and did get stuff.

[18:17:31 CDT(-0500)] <dmccallum54> ahhh the famous "get stuff"

[18:17:47 CDT(-0500)] <GregElliott> elementary.

[18:18:18 CDT(-0500)] <JasonElwood> I see the 688 error primarily in report and MyGPS. I have been seeing it some today in SSP. SSP seems to handle timeouts pretty well as it directs a re-login

[18:18:38 CDT(-0500)] <GregElliott> and I think he had also done a clean on both at some point today. eh mentioend he doesn't typically do the clean

[18:18:39 CDT(-0500)] <JasonElwood> You are trying to access a resource or some message like that

[18:18:50 CDT(-0500)] <dmccallum54> yeah, those aren't the 688s tho

[18:18:58 CDT(-0500)] <dmccallum54> 688s are catastrophic

[18:19:17 CDT(-0500)] <dmccallum54> or at least they were when i was seeing them

[18:19:41 CDT(-0500)] <GregElliott> yes, when I was producing them, the first wouldn't kill everything, but clicking a few more times did lock up the browser totally.

[18:19:57 CDT(-0500)] <dmccallum54> producing which? the 688s?

[18:20:41 CDT(-0500)] <JasonElwood> I've received the 500 attempting to run a report, clicked back which refreshes the report portlet to the first report in the list, then it will run. only a few times have I received the error in that situation more than once.

[18:20:41 CDT(-0500)] <GregElliott> yes, 688s. ie, after dring the first, and 2nd one, I'd see it hitting the code in KSA

[18:20:52 CDT(-0500)] <JasonElwood> with SSP and MyGPS is pretty much kills the session

[18:21:05 CDT(-0500)] <GregElliott> that, though might be that there was only 1 min between ksa inteval and uP timeout.

[18:21:52 CDT(-0500)] <JasonElwood> Is it too late to re-write uPortal in Sencha?

[18:25:01 CDT(-0500)] <GregElliott> I don't even know where Sencha is. (tongue)

[18:25:09 CDT(-0500)] <GregElliott> I've tried google maps to no avail.

[18:26:07 CDT(-0500)] <dmccallum54> too late for 1.2

[18:27:20 CDT(-0500)] <JasonElwood> 1.2.X then

[18:32:49 CDT(-0500)] <dmccallum54> it's gotta happen

[18:33:59 CDT(-0500)] <GregElliott> JasonElwood are the reports that you have been running as lengthy as Jim's?

[18:34:08 CDT(-0500)] <GregElliott> ie, take a while to run?

[18:36:53 CDT(-0500)] <dmccallum54> TonyUnicon1 i dont think https://github.com/Jasig/SSP/commit/1447ae013bcd4e8b836a7447c28db0be80def425 is going to work

[18:37:14 CDT(-0500)] <dmccallum54> it's still forcing an appointment

[18:37:22 CDT(-0500)] <TonyUnicon1> yes I Know

[18:37:27 CDT(-0500)] <dmccallum54> ok. sorry.

[18:37:34 CDT(-0500)] <TonyUnicon1> heh im sorry

[18:37:55 CDT(-0500)] <TonyUnicon1> im working feverishly to correct it

[18:38:15 CDT(-0500)] <TonyUnicon1> trying to figure out a way not to clobber an appointment

[18:38:23 CDT(-0500)] <TonyUnicon1> which is what I think that code was there to do

[18:38:26 CDT(-0500)] <dmccallum54> yeah

[18:39:35 CDT(-0500)] <TonyUnicon1> if I had realized this sooner I wouldn't have piggybacked on that API call

[18:39:37 CDT(-0500)] <JasonElwood> no. definitely not as long. he's also testing 100+ records

[18:39:42 CDT(-0500)] <TonyUnicon1> i didn't realize it wasn't always required

[18:40:15 CDT(-0500)] <JasonElwood> normally, the 500 error happens pretty immediately