Review Request 23329: remove embedded jquery

2014-07-07 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/ --- Review request for Aurora, David McLaughlin and Brian Wickman. Bugs: AURORA-578

Re: Review Request 23329: remove embedded jquery

2014-07-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review47428 --- src/main/python/apache/thermos/observer/http/templates/filebrowse.t

Re: Review Request 23329: remove embedded jquery

2014-07-09 Thread David Robinson
> On July 8, 2014, 1:18 a.m., Kevin Sweeney wrote: > > src/main/python/apache/thermos/observer/http/templates/filebrowse.tpl, line > > 62 > > > > > > -1, this is pretty much unacceptable from a security standpoint - da

Re: Review Request 23329: remove embedded jquery

2014-07-09 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/ --- (Updated July 9, 2014, 9:50 p.m.) Review request for Aurora, David McLaughlin a

Re: Review Request 23329: remove embedded jquery

2014-07-29 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review49045 --- Reviewers - ping? - Bill Farner On July 9, 2014, 9:50 p.m., David

Re: Review Request 23329: remove embedded jquery

2014-07-29 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review49084 --- Not sure I understand the motivation here. It's generally best to ho

Re: Review Request 23329: remove embedded jquery

2014-07-30 Thread David Robinson
> On July 30, 2014, 4:25 a.m., David McLaughlin wrote: > > Not sure I understand the motivation here. It's generally best to host your > > own static assets. Users typically debug their Aurora jobs by looking at individual instances, which usually involves accessing the web interfaces (provide

Re: Review Request 23329: remove embedded jquery

2014-07-30 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review49153 --- ./src/main/python/apache/thermos/observer/http/assets/jquery* should

Re: Review Request 23329: remove embedded jquery

2014-09-29 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review54897 --- Can this be discarded? - Kevin Sweeney On July 9, 2014, 2:50 p.m.

Re: Review Request 23329: remove embedded jquery

2014-10-23 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review58218 --- This patch does not apply cleanly on master (53f4e73), do you need t