search

adlnet-archive / 3D-Repository

The ADL 3D Repository is a website for uploading, finding, and downloading 3D models.
25 stars 16 forks source link

Learning Registry Integration fails #17

Open rchadwic opened 12 years ago

rchadwic commented 12 years ago

Something has changed in the LR, and the publish to the lr2 test node is not working. This is causing a much greater crash than it should, and killing the app pool. This needs to fail more gracefully.

wegrata commented 12 years ago

The new code requires auth on submit. You can go to http://sandbox.learningregistry.org/auth to create an account

On Fri, Sep 7, 2012 at 1:42 PM, rchadwic notifications@github.com wrote:

Something has changed in the LR, and the publish to the lr2 test node is not working. This is causing a much greater crash than it should, and killing the app pool. This needs to fail more gracefully.

— Reply to this email directly or view it on GitHubhttps://github.com/adlnet/3D-Repository/issues/17.

wegrata commented 12 years ago

Not sure what version of .NET the 3dr currently requires but if you can use the async/await stuff to do the post it may help, or look at the threadpool class.

On Fri, Sep 7, 2012 at 1:45 PM, Walt Grata wegrata@gmail.com wrote:

The new code requires auth on submit. You can go to http://sandbox.learningregistry.org/auth to create an account

On Fri, Sep 7, 2012 at 1:42 PM, rchadwic notifications@github.com wrote:

Something has changed in the LR, and the publish to the lr2 test node is not working. This is causing a much greater crash than it should, and killing the app pool. This needs to fail more gracefully.

— Reply to this email directly or view it on GitHubhttps://github.com/adlnet/3D-Repository/issues/17.

rchadwic commented 12 years ago

It actually fires off a thread and forgets about it, which is nice because it does not hold up processing. However, somehow the error in that thread is resetting the app pool, which makes the page load super slow, and the app pool has to restart each time. I've just disabled the integration in the config settings for now - have to look into this next week. It's important that the site be fast and stable since we might get a lot of traffic from that article.