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

500 Error when following steveklabnik@rstat.us #698

Closed
kytrinyx opened this issue Oct 22, 2012 · 12 comments
Closed

500 Error when following steveklabnik@rstat.us #698

kytrinyx opened this issue Oct 22, 2012 · 12 comments
Assignees

Comments

@kytrinyx
Copy link

I entered steveklabnik@rstat.us on a whim into the follow form, and the first time this happened I got the standard rails 500 error. The next couple of times I tried I got a timeout.

Strangely, I ended up with 1 following even though I hadn't followed anyone yet at the time.

On a lark I tried adding nobody@rstat.us, and got a nice error message back explaining that this wasn't going to be possible.

On second thought, maybe I shouldn't be trying to follow someone locally using the remote link. PEBKAC, perhaps?

I'm completely unfamiliar with this project at the moment, so I can't even guess as to whether this is related to other issues (#596 or #565, maybe?).

@tgraham
Copy link
Member

tgraham commented Oct 22, 2012

Not sure when you tried doing the follow stuff, but it might be related to the Amazon US-East problems going on since rstat.us is deployed to Heroku.

Then again, it could totally be something else.

@steveklabnik
Copy link
Contributor

This bug is of the highest priority!!!!!!

@ghost
Copy link

ghost commented Oct 22, 2012

+1 on this bug being critical.

@rkh
Copy link

rkh commented Oct 22, 2012

Why do you wanna follow him? I hear he's using app.net now.

@steveklabnik
Copy link
Contributor

@LindseyB
Copy link
Contributor

lololololol

@carols10cents
Copy link
Contributor

-1 on this bug being critical-- he hasn't even posted in over a month.

Also, I think this is a duplicate of #565 as you said, @kytrinyx. We should totally make this work just fine.

Closing #565 as a duplicate of this one, though, since this one is more hilarious.

@steveklabnik
Copy link
Contributor

he hasn't even posted in over a month.

If you follow me on twitter, that can be construed as a feature.

@wilkie
Copy link
Contributor

wilkie commented Oct 23, 2012

he hasn't even posted in over a month.

If you follow me on twitter, that can be construed as a feature.

It's true. Otherwise, we would have seen a surge of people request server-side filters. :)

@ghost ghost assigned gavinlaking Oct 25, 2012
@gavinlaking
Copy link
Contributor

I'm taking this opportunity to refactor some of the code to help us solve this one. I've also thrown my email address at it, uncovering a 404 error from what I believe to be OStatus::Feed. I'll have more later.

@ghost ghost assigned gavinlaking Oct 28, 2012
@gavinlaking
Copy link
Contributor

I've extracted this out to a couple of services, and tidied up the tests surrounding this which should give more insight and better ability to handle potential errors in this area. I'll be creating a pull request shortly.

carols10cents pushed a commit to carols10cents/rstat.us that referenced this issue Nov 17, 2012
… a local user that doesn't exist using the remote form)
carols10cents added a commit to carols10cents/rstat.us that referenced this issue Nov 17, 2012
…ting users when domain entered is the current one.
@carols10cents
Copy link
Contributor

I believe everything for this has been merged in, so I'm closing this issue. Thank you for all your work, @gavinlaking!!!! ❤️ ❤️ ❤️

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

No branches or pull requests

8 participants