Greg, So are some services broken when we have the period? Can you identify them?
Chris > On 11-08-08 01:57 PM, Greg Logan wrote: > > Hi Folks, > > > > I noticed and filed http://opencast.jira.com/browse/MH-8049 this > > morning, and upon further investigation this appears to be caused > > by the change committed as a proposed fix for > > http://opencast.jira.com/browse/MH-7816. The endpoint MH-8049 > > requires looked like /capture-admin/agents/agents.xml (or .json, I > > don't remember which). The changeset committed for MH-7816 removes > > the period from the endpoint, which changes the endpoint > > to /capture-admin/agents/agentsxml. > > Bah, it adds it rather than removes. The endpoint now *requires* the > period and type (json/xml), something I was trying to avoid when > writing the optional parameters. This is also why the admin UI > breaks: it's calling for capture-admin/agents. > > G > > > I don't think this is the right way to go about this, and changing > > REST endpoints this late in the 1.2 release cycle is an especially > > bad idea. Since these periods are used extensively in other > > services we should instead we should be focusing on making the REST > > doc generation code understand and process these optional > > parameters. > > > > Thoughts? I think reverting that changeset is the best option at > > this point... > > G > > > > > > > > > > _______________________________________________ > > Matterhorn mailing list > > [email protected] > > http://lists.opencastproject.org/mailman/listinfo/matterhorn > > > > > > To unsubscribe please email > > [email protected] > > _______________________________________________ > > -- Christopher Brooks, BSc, MSc ARIES Laboratory, University of Saskatchewan Web: http://www.cs.usask.ca/~cab938 Phone: 1.306.966.1442 Mail: Advanced Research in Intelligent Educational Systems Laboratory Department of Computer Science University of Saskatchewan 176 Thorvaldson Building 110 Science Place Saskatoon, SK S7N 5C9 _______________________________________________ Matterhorn mailing list [email protected] http://lists.opencastproject.org/mailman/listinfo/matterhorn To unsubscribe please email [email protected] _______________________________________________
