Cannot serialize AnonymousUser object using Django Rest Framework - missing attributes

Question

I am trying to make available the current user object so my AJAX front-end can determine if the user is logged in or not.

The current user is either an instance of dajngo.contrib.auth.User for logged in people or AnonymousUser for non-logged in people. The serializer is as follows,

class UserSerializer(serializers.HyperlinkedModelSerializer):
    comments = serializers.HyperlinkedRelatedField(many = True, view_name = 'comment-detail')
    class Meta:
        model = User
        fields = ('url', 'username', 'comments', 'first_name', 'last_name')

with the following view

class CurrentUserView(APIView):
    '''
    Returns the current user, logged in or not
    '''
    def get(self, request, *args, **kwargs):
        serializer = serializers.UserSerializer(request.user)
        return Response(serializer.data)

This serializer works fine for logged in users i.e. it patches on the 'comments' field and sends it off. However for AnonymousUser it chokes. It claims that there is no method 'comments' (which there isn't but it should be patched on right?) and if I remove the 'comments' requirement from fields it then complains about no 'first_name'.

Is this an issue with AnonymousUser not being an instance of django.contrib.auth.User? If so, how do I remedy it?

I realise I could check at the View level and return a custom data object if it is an instance of AnonymousUser but is there a neater way to do this?

Ctrl-C · Accepted Answer · 2013-11-23 16:27:54Z

1

It's better to serialize like this, moving authentication check to backend (where it belongs):

{
    'is_logged_in': True,
    'user': <<serialized user>>,
}

and:

{
    'is_logged_in': False,
    'user': None, # or something else like this
}

It's common solution in many APIs. Metadata is on top of JSON and possible objects under a key(s). You can check if user is AnonymousUser or User by checking .is_authenticated() - it's been specially created for such purpose (compare with: AnonymousUser).

answered Nov 23, 2013 at 16:27

Ctrl-C

4,1721 gold badge28 silver badges30 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Dovydas Šopa · Accepted Answer · 2016-05-02 06:43:44Z

0

Modify your view like shown below

class CurrentUserView(APIView):
    '''
    Returns the current user, logged in or not
    '''
    def get(self, request, *args, **kwargs):
        if request.user.is_authenticated():
            serializer = serializers.UserSerializer(request.user)
            return Response(serializer.data)
        else:
            return Response([])`

edited May 2, 2016 at 6:43

Dovydas Šopa

2,4008 gold badges28 silver badges34 bronze badges

answered May 2, 2016 at 6:00

Krishna G Nair

2682 silver badges3 bronze badges

Collectives™ on Stack Overflow

Cannot serialize AnonymousUser object using Django Rest Framework - missing attributes

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related