Getting the request.user form pre_save in Django

Question

I have a global pre_save signal where I need to get the current logged in user i.e request.user.

I have tried the following which seems to work however, it looks like a bad idea...

@receiver(pre_save)
def my_handler(sender, **kwargs):
    import inspect

    for frame_record in inspect.stack():
        if frame_record[3] == 'get_response':
            request = frame_record[0].f_locals['request']
            break
        else:
            request = None
    print(request.user) <--- correct user is given

Are there any potential issues with above? Is there a better way to achieve this?

Using Python 3.4 and Django 1.8.4

did you tried debugging the code if request is there or not? — Ajay Gupta, Sep 01 '15 at 13:14
@AjayGupta No but this works but can I rely on it? If not are there better options? — Prometheus, Sep 01 '15 at 13:23
Potential issues are, when you create model from Django shell, from management command or any other method. — Yaroslav Admin, Sep 01 '15 at 13:24
Also take a look at [this question](http://stackoverflow.com/questions/2414473/how-to-assign-currently-logged-in-user-as-default-value-for-a-model-field). — Yaroslav Admin, Sep 01 '15 at 13:25
@Falcon1 this is a good option, i don't see any better way to do this — Ajay Gupta, Sep 01 '15 at 13:29

score 4 · Accepted Answer · answered Sep 01 '15 at 13:24

4

I think middleware will be better solution. Check this project to get the idea django-cuser.

btw. Inspecting stack to get the request is brilliant example of how can python be abused and still solve problems :)

answered Sep 01 '15 at 13:24

beezz

2,398
19
15

Getting the request.user form pre_save in Django

1 Answers1