Page Comparison

[08:27:19 EDT(-0400)] * anastasiac (n=team@142.150.154.105) has joined ##uportal
[08:58:23 EDT(-0400)] * colinclark (n=colin@bas1-toronto09-1279475430.dsl.bell.ca) has joined ##uportal
[09:20:06 EDT(-0400)] * esm (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[09:44:20 EDT(-0400)] * EiNZTEiN (n=einztein@205.241.143.4) has joined ##uportal
[09:44:54 EDT(-0400)] * EiNZTEiN_ (n=einztein@205.241.143.4) has joined ##uportal
[09:54:38 EDT(-0400)] * dstn (n=dstn@unaffiliated/dstn) has joined ##uportal
[09:58:16 EDT(-0400)] * EricDalquist (n=dalquist@bohemia.doit.wisc.edu) has joined ##uportal
[10:09:31 EDT(-0400)] * colinclark (n=colin@142.150.154.101) has joined ##uportal
[10:10:26 EDT(-0400)] * michelled (n=team@142.150.154.197) has joined ##uportal
[10:48:04 EDT(-0400)] * athena7 (n=athena7@lumina.its.yale.edu) has joined ##uportal
[11:56:21 EDT(-0400)] * holdorph (n=holdorph@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[12:19:23 EDT(-0400)] <athena7> what sets the remoteUser in the portlet request in up3?
[12:19:36 EDT(-0400)] <EricDalquist> ah I think the portal request wrapper
[12:19:59 EDT(-0400)] <athena7> hm, ok, i'll have to look at that
[12:20:02 EDT(-0400)] <EricDalquist> yup
[12:20:02 EDT(-0400)] <EricDalquist> PortalHttpServletRequest
[12:20:12 EDT(-0400)] <EricDalquist> that handles remote user, user principal, user in role
[12:20:38 EDT(-0400)] <EricDalquist> all using the IPersonManager configured in the spring context
[12:22:25 EDT(-0400)] <athena7> thanks
[12:23:08 EDT(-0400)] <athena7> oh yes, i see it
[12:23:10 EDT(-0400)] <athena7> perfect, thanks!
[12:23:20 EDT(-0400)] <EricDalquist> yup
[12:23:22 EDT(-0400)] <athena7> more little fixes to accomodate this pretend guest behavior
[12:23:23 EDT(-0400)] <athena7> bah
[12:23:25 EDT(-0400)] <EricDalquist> ah
[12:23:33 EDT(-0400)] <EricDalquist> would it be better to put it in IPersonManager?
[12:23:39 EDT(-0400)] <athena7> i found a bug in the merginguserinfoservice, incidentally
[12:23:40 EDT(-0400)] <EricDalquist> or a IPersonManager wrapper?
[12:23:43 EDT(-0400)] <EricDalquist> neat
[12:23:52 EDT(-0400)] <EricDalquist> can you just create a jira for it so folks know about it?
[12:24:20 EDT(-0400)] <athena7> yeah i will
[12:24:36 EDT(-0400)] <athena7> it dies if one of the userinfo services returns null
[12:24:39 EDT(-0400)] <athena7> rather than an empty map
[12:25:11 EDT(-0400)] <EricDalquist> ah
[12:25:19 EDT(-0400)] <athena7> woops.
[12:25:31 EDT(-0400)] <EricDalquist> oh well
[12:25:33 EDT(-0400)] <EricDalquist> easy fix
[12:25:45 EDT(-0400)] <athena7> yeah
[12:25:49 EDT(-0400)] <athena7> ok, i'm going to go grab lunch
[12:25:49 EDT(-0400)] <athena7> bbl
[12:25:55 EDT(-0400)] <EricDalquist> later
[12:35:12 EDT(-0400)] * holdorph (n=holdorph@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[12:39:37 EDT(-0400)] * esm_ (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[12:49:33 EDT(-0400)] * EiNZTEiN_ (n=einztein@205.241.143.4) has joined ##uportal
[13:03:49 EDT(-0400)] <athena7> so Eric it looks like the PortalHttpServletRequest.getUserPrincipal() method returns null if the person is a guest?
[13:03:56 EDT(-0400)] <EricDalquist> yup
[13:04:10 EDT(-0400)] <EricDalquist> we may need to revisit where things are done
[13:04:27 EDT(-0400)] <EricDalquist> the remote-user and user-principal methods follow the portlet requirements
[13:04:35 EDT(-0400)] <EricDalquist> which return null for guest
[13:04:55 EDT(-0400)] <EricDalquist> if other portal code is going to need some other behavior we'll just have to move that behavior into the portlet request wrapper
[13:05:21 EDT(-0400)] <athena7> the portlet spec requires them to be null for the guest user?
[13:05:30 EDT(-0400)] <EricDalquist> I believe so
[13:06:04 EDT(-0400)] <athena7> that seems really weird
[13:06:23 EDT(-0400)] <athena7> i can imagine a portlet wanting information on the guest user
[13:06:27 EDT(-0400)] <athena7> full name or something random
[13:07:11 EDT(-0400)] <EricDalquist> hrm
[13:07:15 EDT(-0400)] <EricDalquist> actually looking at the spec
[13:07:25 EDT(-0400)] <EricDalquist> it doesn't require that for remoteuser or user principal
[13:07:32 EDT(-0400)] <athena7> hm ok
[13:07:37 EDT(-0400)] <EricDalquist> just that the user-info map must be null for un-authd users
[13:07:47 EDT(-0400)] <athena7> oh, argh
[13:07:49 EDT(-0400)] <athena7> that's actually worse
[13:07:57 EDT(-0400)] <EricDalquist> well in your case
[13:08:02 EDT(-0400)] <EricDalquist> of a partially authed user
[13:08:07 EDT(-0400)] <EricDalquist> they aren't 'un-authenticated'
[13:08:14 EDT(-0400)] <EricDalquist> they are just loosely authenticated
[13:08:20 EDT(-0400)] <athena7> yeah, i guess that's true
[13:08:30 EDT(-0400)] <athena7> we called them "identified" for convenience's sake
[13:08:31 EDT(-0400)] <EricDalquist> though, you're going to have fun with portlet entity/window issues
[13:08:55 EDT(-0400)] <EricDalquist> unless you somehow switch the userid to that of the loosely authd user
[13:09:17 EDT(-0400)] <athena7> the userid of the IPerson?
[13:09:24 EDT(-0400)] <EricDalquist> yup
[13:09:38 EDT(-0400)] <athena7> yeah, that's how we created the person originally
[13:10:10 EDT(-0400)] <EricDalquist> ah so your loose-auth does set the correct IPerson userid?
[13:10:15 EDT(-0400)] <athena7> yep
[13:10:19 EDT(-0400)] <EricDalquist> that's good
[13:10:37 EDT(-0400)] <EricDalquist> then yeah, those methods should probably continue to return null for guest users
[13:10:48 EDT(-0400)] <EricDalquist> but your loose-auth users are authenticated in some way
[13:11:17 EDT(-0400)] <athena7> they're actually not marked authenticated
[13:11:25 EDT(-0400)] <athena7> we set an "identified" attribute for the user
[13:11:30 EDT(-0400)] <EricDalquist> ah
[13:11:39 EDT(-0400)] <athena7> so we could check for that in the getUserPrincipal method
[13:11:42 EDT(-0400)] <EricDalquist> yup
[13:13:21 EDT(-0400)] <athena7> ok
[13:13:31 EDT(-0400)] <athena7> i just wanted to make sure that wasn't likely to have unintended consequences
[13:13:37 EDT(-0400)] <EricDalquist> I don't think so
[13:14:04 EDT(-0400)] <EricDalquist> I think the in the non-uportal centric use authenticated covers both isAuthenticated and having the identified attribute
[13:14:05 EDT(-0400)] <athena7> what's likely to happen for the preferences editing?
[13:14:21 EDT(-0400)] <athena7> will they get the guest preferences behavior?
[13:14:54 EDT(-0400)] <EricDalquist> um
[13:14:56 EDT(-0400)] <EricDalquist> hrm
[13:15:15 EDT(-0400)] <EricDalquist> you'll need to take a look at PortletPreferencesServiceImpl
[13:15:22 EDT(-0400)] <EricDalquist> which is easy to extend
[13:15:27 EDT(-0400)] <EricDalquist> and override the isGuestUser method
[13:15:58 EDT(-0400)] <athena7> actually we want them to get the guest user behavior
[13:16:06 EDT(-0400)] <athena7> so if that's what would happen that's actually perfect
[13:16:38 EDT(-0400)] <EricDalquist> I think so
[13:16:47 EDT(-0400)] <EricDalquist> you should still take a look at that method to make sure it does what you want
[13:16:56 EDT(-0400)] <EricDalquist> do you want to load any of their preferences?
[13:17:28 EDT(-0400)] <EricDalquist> so if I login, set and pref X
[13:17:32 EDT(-0400)] <EricDalquist> then I come back as identified
[13:17:36 EDT(-0400)] <EricDalquist> should I still see pref X?
[13:19:47 EDT(-0400)] <athena7> yes, loading preferences would be good
[13:19:52 EDT(-0400)] <EricDalquist> that isn't supported
[13:20:11 EDT(-0400)] <EricDalquist> and looking at the pref service there isn't an easy way to do it without modifying code
[13:20:15 EDT(-0400)] <athena7> wait so what happens for portlets on the guest pages?
[13:20:22 EDT(-0400)] <EricDalquist> if you are a guest
[13:20:32 EDT(-0400)] <EricDalquist> no preferences from the IPortletEntity are loaded
[13:20:37 EDT(-0400)] <athena7> ack
[13:20:42 EDT(-0400)] <EricDalquist> instead a temporary Map is created in the users session
[13:20:50 EDT(-0400)] <athena7> so we couldn't have the weather portlet on the guest page or anything like that?
[13:21:05 EDT(-0400)] <EricDalquist> not if it uses prefs for storing settings
[13:21:11 EDT(-0400)] <EricDalquist> now it wouldn't be hard to change
[13:21:17 EDT(-0400)] <EricDalquist> but it would require modifying code
[13:21:23 EDT(-0400)] <EricDalquist> instead of simply extending it
[13:21:31 EDT(-0400)] <EricDalquist> take a look at PortletPreferencesServiceImpl
[13:21:42 EDT(-0400)] <athena7> ok
[13:21:48 EDT(-0400)] <athena7> that's going to be a major issue for yale
[13:22:04 EDT(-0400)] <EricDalquist> well
[13:22:16 EDT(-0400)] <EricDalquist> we can work out some modification to the class to include in 3.0.1
[13:22:23 EDT(-0400)] <EricDalquist> and you can use that mod and extend it
[13:22:30 EDT(-0400)] <athena7> yeah
[13:22:34 EDT(-0400)] <EricDalquist> but that class will need to be modified
[13:23:22 EDT(-0400)] <athena7> ok
[13:23:29 EDT(-0400)] <athena7> that's actually a difference from up26
[13:24:03 EDT(-0400)] <EricDalquist> yeah ... there are bound to be a few since the portlet support was a from-scratch rewrite
[13:24:13 EDT(-0400)] <athena7> yeah
[13:24:37 EDT(-0400)] <EricDalquist> so some ideas are:
[13:25:05 EDT(-0400)] <EricDalquist> add two new methods isGuestStore and isGuestLoad which by default both delegate to isGuest
[13:25:28 EDT(-0400)] <EricDalquist> then you can determine if a user is a guest conditionally based on if it is a load or a store
[13:25:44 EDT(-0400)] <EricDalquist> though that would make changes not stick
[13:25:49 EDT(-0400)] <EricDalquist> which may be desired
[13:26:09 EDT(-0400)] <athena7> yes
[13:26:17 EDT(-0400)] <EricDalquist> I guess it would be good if you could look through that class and come back with your requirements
[13:26:22 EDT(-0400)] <EricDalquist> right now guests can change prefs
[13:26:25 EDT(-0400)] <athena7> i think the desired behavior is to load preferences for the guest user, but not let users change them
[13:26:27 EDT(-0400)] <EricDalquist> and they stick until their sesson expires
[13:26:35 EDT(-0400)] <athena7> yeah
[13:26:41 EDT(-0400)] <athena7> so the requirements would be
[13:26:45 EDT(-0400)] <athena7> load preferences as guest
[13:27:23 EDT(-0400)] <athena7> and if users want to muck around with them for their session, that's fine, although not required, but definitely don't persist the changed preferences or share them with other users
[13:27:47 EDT(-0400)] <athena7> i imagine this will likely be a blocker for yale
[13:28:03 EDT(-0400)] <EricDalquist> ok
[13:28:24 EDT(-0400)] <EricDalquist> so that first idea doesn't cover the 'let them muck with them for there session'
[13:29:03 EDT(-0400)] <athena7> they really don't need to
[13:29:14 EDT(-0400)] <athena7> i think yale would probably prefer that users not get to modify guest user preferences at alll
[13:29:27 EDT(-0400)] <athena7> but if they could it would be ok, as long as it wouldn't affect anyone else
[13:29:57 EDT(-0400)] <EricDalquist> I'm trying to think of the most flexible option here
[13:30:16 EDT(-0400)] <athena7> yeah
[13:31:16 EDT(-0400)] <EricDalquist> my best idea so far would be to remove the isGuestUser method and replace it with isStoreInEntity(), isStoreInMemory(), isLoadFromEntity(), isLoadFromMemory()
[13:31:25 EDT(-0400)] <EricDalquist> all of those would take the current HttpServletRequest as an argument
[13:31:38 EDT(-0400)] <athena7> gotcha
[13:31:40 EDT(-0400)] <EricDalquist> and would allow folks to sub-class the service
[13:31:43 EDT(-0400)] <EricDalquist> and provide their own logic
[13:32:00 EDT(-0400)] <EricDalquist> think you'd be comfortable coding up the patch?
[13:32:01 EDT(-0400)] <athena7> that sounds pretty reasonable, i think i like that plan
[13:32:06 EDT(-0400)] <athena7> yes
[13:32:11 EDT(-0400)] <athena7> but probably not this week
[13:32:14 EDT(-0400)] <EricDalquist> no problem
[13:32:29 EDT(-0400)] <athena7> i think i'm about to enter panicked-too-much-to-do mode
[13:32:46 EDT(-0400)] <EricDalquist> (I know I ask for these a lot but...) could you create a Jira improvement task with a fix-for 3.0.1?
[13:32:47 EDT(-0400)] <EricDalquist> yeah
[13:32:50 EDT(-0400)] <EricDalquist> I'm already there
[13:32:54 EDT(-0400)] <athena7> sure i can do that
[13:32:58 EDT(-0400)] <EricDalquist> trying to get all these presentations read
[13:33:00 EDT(-0400)] <EricDalquist> ready ...
[13:33:02 EDT(-0400)] <athena7> yeah
[13:33:04 EDT(-0400)] <athena7> :/
[13:33:08 EDT(-0400)] <athena7> i haven't had time to start mine
[13:33:14 EDT(-0400)] <EricDalquist> I really don't like writing them
[13:33:17 EDT(-0400)] <athena7> user testing all tomorrow afternoon
[13:34:08 EDT(-0400)] <EricDalquist> ah
[13:34:45 EDT(-0400)] <athena7> thanks for all the help
[13:34:49 EDT(-0400)] <athena7> at least i know what's going on now
[13:34:51 EDT(-0400)] <EricDalquist> np
[13:43:53 EDT(-0400)] <athena7> ok, i've recorded your suggestions in a jira
[13:44:01 EDT(-0400)] <EricDalquist> thanks!
[13:46:38 EDT(-0400)] <athena7> thank you
[14:11:08 EDT(-0400)] * EiNZTEiN (n=einztein@205.241.143.4) has joined ##uportal
[14:15:54 EDT(-0400)] * holdorph (n=holdorph@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[14:24:04 EDT(-0400)] * esm (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[14:25:21 EDT(-0400)] * esm (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[14:26:20 EDT(-0400)] * esm_ (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[14:27:03 EDT(-0400)] * esm (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[14:28:41 EDT(-0400)] * esm_ (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[14:29:50 EDT(-0400)] * esm (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[14:35:12 EDT(-0400)] * esm_ (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[15:46:00 EDT(-0400)] * EricDalquist (n=EricDalq@ppp-70-226-161-218.dsl.mdsnwi.ameritech.net) has joined ##uportal
[15:54:00 EDT(-0400)] * michelled (n=team@142.150.154.197) has left ##uportal
[16:28:06 EDT(-0400)] <athena7> hey Eric, any insight into what "javax.portlet.PortletSecurityException: Secure URLs are not supported at this time" is coming from?
[16:28:24 EDT(-0400)] <EricDalquist> something is calling setSecure on a portlet URL
[16:28:27 EDT(-0400)] <EricDalquist> hrm
[16:28:35 EDT(-0400)] <EricDalquist> that may be a regresion too
[16:28:45 EDT(-0400)] <athena7> this is just logged in normally
[16:28:47 EDT(-0400)] <EricDalquist> uP2 may have just silently ignored that call
[16:29:09 EDT(-0400)] <EricDalquist> so the portlet URL api has a setSecure method when generating URLs
[16:29:26 EDT(-0400)] <EricDalquist> to require the subisquent URL to be 'secure' whatever that happens to mean for the portal
[16:29:32 EDT(-0400)] <EricDalquist> but it is an optional feature
[16:29:42 EDT(-0400)] <EricDalquist> and since we had never defined it I left it un-implemented
[16:29:50 EDT(-0400)] <athena7> any idea what's causing it?
[16:30:03 EDT(-0400)] <EricDalquist> are you in a vanialla uP3 install?
[16:30:15 EDT(-0400)] <EricDalquist> or do you have other portlets installed?
[16:30:44 EDT(-0400)] <athena7> this is the vanilla bookmarks portlet in up3
[16:30:49 EDT(-0400)] <EricDalquist> hrm
[16:30:58 EDT(-0400)] <athena7> over ssl
[16:31:02 EDT(-0400)] <EricDalquist> ah
[16:31:19 EDT(-0400)] <EricDalquist> is the portlet failing to render?
[16:31:28 EDT(-0400)] <athena7> no, it fails whenever preferences are set
[16:31:46 EDT(-0400)] <athena7> if you reload, they appear to have been saved successfully
[16:31:52 EDT(-0400)] <EricDalquist> yeah
[16:32:01 EDT(-0400)] <EricDalquist> it may actually be a pluto bug then .... heh
[16:32:05 EDT(-0400)] <athena7> ok
[16:32:06 EDT(-0400)] <EricDalquist> can you create a jira with the stack trace?
[16:32:08 EDT(-0400)] <athena7> yes
[16:32:11 EDT(-0400)] <EricDalquist> thanks
[16:32:15 EDT(-0400)] <athena7> i have user interface testing on this tomorrow though
[16:32:30 EDT(-0400)] <EricDalquist> I'll take a look at the trace when you get it posted
[16:32:36 EDT(-0400)] <athena7> so i need to decide whether to fail back to our up2 code and try and get the cookie login stuff working there
[16:33:39 EDT(-0400)] <EiNZTEiN> howdy everyone
[16:34:00 EDT(-0400)] <EricDalquist> athena7: ok, I'll get back to you on it as soon as I can
[16:34:13 EDT(-0400)] <EricDalquist> shouldn't take more than a few minutes to figure out where the problem is
[16:34:34 EDT(-0400)] <athena7> thanks eric . . .
[16:34:44 EDT(-0400)] <athena7> i can fix the bugs in the up2 version if i need to
[16:34:49 EDT(-0400)] <athena7> i just need to decide which i'm going to do
[16:35:19 EDT(-0400)] <EiNZTEiN> any xslt gurus here?
[16:35:52 EDT(-0400)] * esm_ (n=esm@207-53-192-198.dynamic-dsl.qis.net) has joined ##uportal
[16:37:55 EDT(-0400)] <EiNZTEiN> I'm trying to make a file upload dialog use the filename as the name variable when no name is entered
[16:38:17 EDT(-0400)] <EiNZTEiN> the upload dialog has 2 variables: the name and the filename
[16:40:46 EDT(-0400)] <esm_> PortletUrlProviderImpl doesn't support it... setSecure that is
[16:40:56 EDT(-0400)] <athena7> yeah, i see the code there now
[16:41:04 EDT(-0400)] <EricDalquist> // Set secure of the redirect URL if necessary.
[16:41:04 EDT(-0400)] <EricDalquist> if (actionRequest.isSecure()) {
[16:41:04 EDT(-0400)] <EricDalquist> redirectURL.setSecure();
[16:41:04 EDT(-0400)] <EricDalquist> }
[16:41:11 EDT(-0400)] <athena7> so i guess setSecure is automatically called if you're using ssl?
[16:41:13 EDT(-0400)] <EricDalquist> that is in PortletContainerImpl
[16:41:29 EDT(-0400)] <EricDalquist> where it generates the redirect URL after the action is complete
[16:41:36 EDT(-0400)] <athena7> ahh
[16:41:37 EDT(-0400)] <EricDalquist> I'm checking to see what up2 did
[16:41:47 EDT(-0400)] <EricDalquist> I'm guessing it just ignored whatever was set
[16:41:53 EDT(-0400)] <esm> oh right yes it would be on an action.
[16:41:57 EDT(-0400)] <esm> for prefs
[16:42:20 EDT(-0400)] <EricDalquist> and ... aparently pluto doesn't use its own api
[16:42:35 EDT(-0400)] <athena7> oh
[16:42:36 EDT(-0400)] <athena7> lol
[16:42:42 EDT(-0400)] <EricDalquist> PortletURLProvider.isSecureSupported() is a method
[16:42:46 EDT(-0400)] <EricDalquist> and we return false
[16:42:48 EDT(-0400)] <esm> b/c it isn't implemented int its url provider.
[16:42:49 EDT(-0400)] <EricDalquist> but pluto never checks
[16:42:56 EDT(-0400)] <athena7> i guess if it came down to it i could just comment out that block of code you copied, if i need to
[16:43:04 EDT(-0400)] <EricDalquist> well that is part of pluto
[16:43:09 EDT(-0400)] <esm> right
[16:43:17 EDT(-0400)] <EricDalquist> athena7: a quick fix would be to edit PortletURLProviderImpl
[16:43:26 EDT(-0400)] <EricDalquist> and comment out line 70
[16:43:30 EDT(-0400)] <esm> wee
[16:43:34 EDT(-0400)] <EricDalquist> not really correct
[16:43:40 EDT(-0400)] <athena7> yeah that makes sense
[16:43:49 EDT(-0400)] <athena7> just so long as it works tomorrow, i don't really care, at the moment
[16:43:58 EDT(-0400)] <EricDalquist> esm: shouldn't the PortletContainerImpl be checking isSecureSupported?
[16:44:38 EDT(-0400)] <EricDalquist> ah I see
[16:44:52 EDT(-0400)] <EricDalquist> they set it blindly and throw an exception if it isn't supported
[16:44:56 EDT(-0400)] <EricDalquist> which is what the spec says to do
[16:44:58 EDT(-0400)] <EricDalquist> :/
[16:44:59 EDT(-0400)] <athena7> ah
[16:45:09 EDT(-0400)] <EricDalquist> not sure what the best action is here ...
[16:45:17 EDT(-0400)] <EricDalquist> we probably need to figure out how to actually support this
[16:45:23 EDT(-0400)] <athena7> yeah
[16:45:25 EDT(-0400)] <EricDalquist> or just ignore it like we did previously
[16:45:32 EDT(-0400)] <athena7> what does it mean to have a secure url anyway?
[16:45:33 EDT(-0400)] <EricDalquist> the hard part is the portal doesn't really know if it can use https
[16:45:35 EDT(-0400)] <athena7> force ssl?
[16:45:37 EDT(-0400)] <EricDalquist> yeah
[16:45:39 EDT(-0400)] <athena7> or does it mean something more than that
[16:45:43 EDT(-0400)] <EricDalquist> so the idea is the portal can run over http
[16:45:55 EDT(-0400)] <EricDalquist> and the portlet can request certain URLs over a secure protocol
[16:45:58 EDT(-0400)] <EricDalquist> https
[16:46:00 EDT(-0400)] <athena7> ok, that makes sense
[16:46:13 EDT(-0400)] <EricDalquist> but how does the portal, running over http, know if it can run over https?
[16:47:13 EDT(-0400)] <athena7> that's a good point . . .
[16:47:15 EDT(-0400)] <EricDalquist> I guess we could say it is supported if the request URL was over https
[16:47:19 EDT(-0400)] <EricDalquist> which would fix this problem here
[16:47:32 EDT(-0400)] <EricDalquist> it would still fail if called when running over http
[16:47:34 EDT(-0400)] <athena7> ok, much better!
[16:47:37 EDT(-0400)] <EricDalquist> but I don't see an easy way around that
[16:47:37 EDT(-0400)] <athena7> thank you so much
[16:47:51 EDT(-0400)] <athena7> clearly i didn't do enough advance planning on this one
[16:48:02 EDT(-0400)] <EricDalquist>
[16:48:14 EDT(-0400)] <athena7> yeah i'm not sure of a failproof way to deal with that either
[16:48:40 EDT(-0400)] <EricDalquist> so change line 63 to:
[16:48:40 EDT(-0400)] <EricDalquist> return this.httpServletRequest.isSecure();
[16:48:44 EDT(-0400)] <athena7> seriously, i really appreciate the emergency help
[16:48:52 EDT(-0400)] <EricDalquist> and the setSecure method to:
[16:48:52 EDT(-0400)] <EricDalquist> if (!this.httpServletRequest.isSecure()) {
[16:48:52 EDT(-0400)] <EricDalquist> throw new PortletSecurityException("Secure URLs are not supported at this time");
[16:48:52 EDT(-0400)] <EricDalquist> }
[16:49:05 EDT(-0400)] <EricDalquist> I, of course haven't tested any of this
[16:49:14 EDT(-0400)] <athena7> that makes sense
[16:49:24 EDT(-0400)] <athena7> i can try it out, and if it works, add that to the jira
[16:49:26 EDT(-0400)] <athena7> or just commit it
[16:49:31 EDT(-0400)] <EricDalquist> that works
[16:49:39 EDT(-0400)] <EricDalquist> feel free to commit it if it works
[16:49:46 EDT(-0400)] <athena7> will do
[16:49:57 EDT(-0400)] <EricDalquist> you can just do it on trunk and I'll svnmerge it over to the branch later
[16:50:04 EDT(-0400)] <athena7> ok
[16:50:08 EDT(-0400)] <EricDalquist> or if your comfortable with svnmerge you can do that too
[16:50:14 EDT(-0400)] <athena7> haven't used it yet
[16:50:20 EDT(-0400)] <EricDalquist> no problem
[16:50:26 EDT(-0400)] <athena7> so yeah i guess we need a up3 maintenance branch, huh
[16:50:32 EDT(-0400)] <EricDalquist> already have one
[16:50:36 EDT(-0400)] <athena7>
[17:09:17 EDT(-0400)] <athena7> ok, i'm going home before i find any other problems and give myself a heart attack
[17:09:23 EDT(-0400)] <EricDalquist>
[17:09:32 EDT(-0400)] <EricDalquist> have a good night
[17:09:44 EDT(-0400)] <athena7> you too
[17:09:49 EDT(-0400)] <athena7> thanks again
[17:11:35 EDT(-0400)] <EiNZTEiN> EricDalquist, is there a good way to make this code (http://uportal.pastebin.com/d687eb2b) into the conditional statement I mentioned earlier?
[17:11:49 EDT(-0400)] <EiNZTEiN> [15:37:54] <EiNZTEiN> I'm trying to make a file upload dialog use the filename as the name variable when no name is entered
[17:11:50 EDT(-0400)] <EiNZTEiN> [15:38:16] <EiNZTEiN> the upload dialog has 2 variables: the name and the filename
[17:12:00 EDT(-0400)] <EricDalquist> not sure ... XSLT isn't my strong suit
[17:12:04 EDT(-0400)] <EiNZTEiN> oh ok
[17:12:19 EDT(-0400)] <EricDalquist> I'd have to go research via google to figure it out
[17:12:29 EDT(-0400)] <EiNZTEiN> Google University FTW
[17:12:30 EDT(-0400)] <EiNZTEiN>
[17:12:49 EDT(-0400)] <EiNZTEiN> that's what I'm doing already, with little luck
[17:56:15 EDT(-0400)] * EiNZTEiN_ (n=einztein@205.241.143.4) has joined ##uportal
[18:17:42 EDT(-0400)] * dstn (n=dstn@unaffiliated/dstn) has left ##uportal
[21:06:49 EDT(-0400)] * EricDalquist (n=EricDalq@ppp-70-226-161-218.dsl.mdsnwi.ameritech.net) has joined ##uportal