Versions Compared

Key

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

...

[11:06:25 CDT(-0500)] <EricDalquist> well that is bad

[11:06:33 CDT(-0500)] <EricDalquist> we have issues with that here

[11:06:36 CDT(-0500)] <EricDalquist> if one of our servers

[11:06:37 CDT(-0500)] <athena> similarly the google APIs are being included via http instead of https

[11:06:49 CDT(-0500)] <EricDalquist> tries to access my.wisc.edu

[11:06:56 CDT(-0500)] <EricDalquist> that server will drop off the network for ~2 minutes

[11:07:10 CDT(-0500)] <athena> ick

[11:07:12 CDT(-0500)] <EricDalquist> yeah

[11:07:24 CDT(-0500)] <EricDalquist> due to the network magic the layer4 load balancer does

[11:07:26 CDT(-0500)] <athena> so i mean i can just write some code that'll allow configuration of whether it should use http or https

[11:07:32 CDT(-0500)] <EricDalquist> so in general from what I've see

[11:07:34 CDT(-0500)] <EricDalquist> seen

[11:07:35 CDT(-0500)] <athena> but also calls to request.isSecure are going to be wrong

[11:07:53 CDT(-0500)] <EricDalquist> it is "very bad practice" for an app to make any sort of network connection back to itself

[11:08:03 CDT(-0500)] <EricDalquist> unless explicitly configured to

[11:08:11 CDT(-0500)] <EricDalquist> but that isSecure thing

[11:08:19 CDT(-0500)] <EricDalquist> that seems like it could be a config option ifneeded

[11:08:24 CDT(-0500)] <EricDalquist> add a flag in portal.properties

[11:08:48 CDT(-0500)] <EricDalquist> and modify the portal http servlet request wrapper to look for the flag and override the actual value if set

[11:08:56 CDT(-0500)] <athena> i guess maybe with our new filter files we can get away from the portlet connecting back to itself without adding a configuration headache

[11:09:08 CDT(-0500)] <EricDalquist> what is it trying to do?

[11:09:09 CDT(-0500)] <athena> so maybe we should just start using those values in portal.properties and some of hte problem will go away

[11:09:17 CDT(-0500)] <athena> just reading in the default data

[11:09:26 CDT(-0500)] <athena> which could be remote but doesn't have to be

[11:09:44 CDT(-0500)] <EricDalquist> ah so the portlet allows for a remote data source?

[11:09:57 CDT(-0500)] <EricDalquist> yeah I'd make it so the default config is read locally

[11:10:09 CDT(-0500)] <athena> and we can set up the default use case to use the portlet overlay

[11:13:20 CDT(-0500)] <EricDalquist> on a positive note ... while fixing the caching stuff

[11:13:36 CDT(-0500)] <EricDalquist> I also realized that we were not letting resource responses set browser headers

[11:13:39 CDT(-0500)] <EricDalquist> so that works now

[11:15:05 CDT(-0500)] <athena> ah (smile)

[11:15:30 CDT(-0500)] <EricDalquist> so now resource responses can actually set the status code

[11:15:39 CDT(-0500)] <EricDalquist> and arbitrary headers

[11:16:32 CDT(-0500)] <athena> yay!

[14:08:24 CDT(-0500)] <EricDalquist> ok ... just merged in all that caching refactoring

[14:08:31 CDT(-0500)] <EricDalquist> let me know if you see any weirdness athena

[14:08:37 CDT(-0500)] <EricDalquist> but I think it is pretty well tested

[15:49:34 CDT(-0500)] <b-sure> hello uMobile devs. Do you know if there is a specific configuration you need to make to enable the mobile view of the NewsReaderPortlet? I'm using an overlay of the 3.0.2 version.

[15:50:03 CDT(-0500)] <b-sure> this would be viewing the portlet through uMobile

[15:50:30 CDT(-0500)] <b-sure> for some reason, its displaying in the normal view

[15:52:37 CDT(-0500)] <athena> hm

[15:52:42 CDT(-0500)] <athena> are you getting the mobile view for other stuff?

[15:53:09 CDT(-0500)] <b-sure> only for the directory porltet

[15:53:26 CDT(-0500)] <b-sure> othewise everything looks like a shrunken portlet

[15:54:08 CDT(-0500)] <athena> do you by chance have custom theme names?

[15:54:09 CDT(-0500)] <b-sure> I'm not sure if maybe something in the db data may be causing this. or maybe a config chagne

[15:54:19 CDT(-0500)] <b-sure> I think it is DLMXHTML

[15:54:26 CDT(-0500)] <b-sure> without the space?

[15:54:41 CDT(-0500)] <athena> do you have the mobile theme configured? and is that theme showing up when you go to the portal from a mobile browser?

[15:54:43 CDT(-0500)] <b-sure> oh we do have our custom uofchicago theme

[15:54:56 CDT(-0500)] <athena> the portlets are looking at the theme name

[15:55:01 CDT(-0500)] <b-sure> I don't think I have the mobile theme configured

[15:55:10 CDT(-0500)] <athena> ok, you need to do that to use umobile (smile)

[15:55:14 CDT(-0500)] <athena> it's in the default data set

[15:55:19 CDT(-0500)] <athena> but perhaps that didn't get imported?

[15:55:21 CDT(-0500)] <b-sure> muniversality?

[15:55:23 CDT(-0500)] <athena> yes

[15:55:25 CDT(-0500)] <athena> exactly

[15:55:38 CDT(-0500)] <athena> then if the user is using the muniversality theme, the protlets will send the mobile views

[15:55:41 CDT(-0500)] <b-sure> ok maybe we don't have a theme for that yet. is there a default mobile theme

[15:55:46 CDT(-0500)] <athena> yes

[15:55:52 CDT(-0500)] <athena> muniversality would be it

[15:56:09 CDT(-0500)] <athena> it should be pretty obvious if it's configured - when you go to the main portal page you'd get that blue home screen with icons

[15:56:13 CDT(-0500)] <athena> rather than seeing all the portlets

[15:56:13 CDT(-0500)] <b-sure> how does the user enable muniversality?

[15:57:01 CDT(-0500)] <b-sure> well i do see the icons for the different portlets when uMObile is turned on, it when I click on the icon, I get the portlet view instead of the mobile view

[15:57:29 CDT(-0500)] <athena> ok

[15:57:36 CDT(-0500)] <athena> that sounds like the theme is enabled

[15:58:12 CDT(-0500)] <athena> so each of the mobile-enabled portlets has some sort of theme resolver

[15:58:20 CDT(-0500)] <b-sure> ok

[15:58:25 CDT(-0500)] <athena> some have more than one - you want the version that looks at the theme name

[15:58:39 CDT(-0500)] <b-sure> yeah I think the theme resolver for news reader portlet needs to be adjusted.

[15:58:52 CDT(-0500)] <athena> you might set a breakpoint to make sure that implementation is being used, and see if maybe there's a mismatch between the default expected mobile theme name and what you have registered at uchicago

[15:58:58 CDT(-0500)] <athena> it does work out of the box

[15:59:14 CDT(-0500)] <athena> but if you had a custom fname for your theme or something that would require some configuration

[15:59:51 CDT(-0500)] <b-sure> ok. maybe our custom theme is getting in the way. so we have a universality->uofchicago theme for the regular browser, do we need a muniversality->uofchicago for the mobile theme?

[16:00:38 CDT(-0500)] <b-sure> I don't think we have made any mods to any of the mobile theme code for uPortal.

[16:01:36 CDT(-0500)] <b-sure> well anyway. I'll have some more time to work with this tomorrow. thanks for your info athena.

[16:02:26 CDT(-0500)] <athena> sorry, doing a bunch of things (smile)

[16:02:42 CDT(-0500)] <athena> you shouldn't need changes if you have the default name, but i'd take a look with a debugger to track down exactly what's going on

[16:03:07 CDT(-0500)] <b-sure> ok. will do. thanks.