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

fix panic on duplicate remoteuser key #166

Open
wants to merge 1 commit into
base: develop
from

Conversation

@robjloranger
Copy link
Member

commented Aug 9, 2019

this changes handleFetchCollectionInbox to log all errors after
attempting to insert an actor in the remoteusers table. previously
checking for all errors except duplicate keys would cause a panic if
an actor made a request to follow while already having followed.


  • I have signed the CLA

closes #159

fix panic on duplicate remoteuser key
this changes handleFetchCollectionInbox to log _all_ errors after
attempting to insert an actor in the remoteusers table. previously
checking for all errors _except_ duplicate keys would cause a panic if
an actor made a request to follow while already having followed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.