Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Learning Registry Integration fails #17

Open
rchadwic opened this issue Sep 7, 2012 · 3 comments
Open

Learning Registry Integration fails #17

rchadwic opened this issue Sep 7, 2012 · 3 comments

Comments

@rchadwic
Copy link
Contributor

rchadwic commented Sep 7, 2012

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
Copy link
Contributor

wegrata commented Sep 7, 2012

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 [email protected] 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//issues/17.

@wegrata
Copy link
Contributor

wegrata commented Sep 7, 2012

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 [email protected] 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 [email protected] 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//issues/17.

@rchadwic
Copy link
Contributor Author

rchadwic commented Sep 7, 2012

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants