639947 - node assignment should not always proxy to sreg

Status: RESOLVED DUPLICATE of bug 636514

(Cloud Services :: Server: Registration, defect)

Description

[Pete Fritchman [:petef]]

Turning this comment into a bug:

    def user_node(self, request):
        """Returns the storage node root for the user"""
        # XXX if the user has already a node, we should not proxy

This will make life better if the ldap master is down and people are trying to add new devices. It also translates to less ldap master query load (and slightly increased slave query load, which is OK).

Comment 1

[Toby Elliott [:telliott]]

I believe that this bug should be more specifically "please rip all references to proxying out of sreg" and make sure we're using the mozilla core auth module, which should do this correctly.

Comment 2

[Toby Elliott [:telliott]]

er, s/out of sreg/out of user controller/

Comment 3

[Tarek Ziadé (:tarek)]

Yes, that part is going away as soon as 636514 is done