Versions Compared

Key

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

...

[14:26:51 CDT(-0500)] <dmccallum54> this has, um, catastrophic effects on SSP's attempts to look up uP users from its main portlet

[14:27:04 CDT(-0500)] <dmccallum54> git blame instructed me to check with you

[14:27:40 CDT(-0500)] <dmccallum54> if i just remove that restriction, what are the actual changes of either 1) OOM and/or 2) breaking a UI somewhere

[14:27:50 CDT(-0500)] <athena> yes, you can totally just remove that if you like

[14:27:54 CDT(-0500)] <dmccallum54> thx. done

[14:27:57 CDT(-0500)] <athena> basically that's in there for the directory portlet

[14:28:25 CDT(-0500)] <athena> so that you can't enter a search like "a" and get back 5000 results from the university directory

[14:28:38 CDT(-0500)] <athena> to prevent people from just scraping it, as well as for performance reasons

[14:28:52 CDT(-0500)] <dmccallum54> sure sure

[14:28:58 CDT(-0500)] <athena> actually a lot of LDAP servers have a limit for results anyway, so a search like that really would probably cause an error

[14:29:00 CDT(-0500)] <athena> anyway.

[14:29:05 CDT(-0500)] <athena> that really needs to become configurable

[14:29:21 CDT(-0500)] <dmccallum54> k

[14:29:30 CDT(-0500)] <athena> such that schools can turn that on and off to meet their needs

[14:29:34 CDT(-0500)] <athena> feel free to submit a ticket for that