Blockers

[Hiemanshu Sharma]

You added a return $uid line?

- Hiemanshu

On 04/22/2010 02:17 AM, Simon Birtwistle wrote:
> It was a simple 1 line change on line 53 of pnuserapi.php
>    
>> -----Original Message-----
>> From: Robyn Bergeron [mailto:robyn.bergeron at gmail.com]
>> Sent: 21 April 2010 17:03
>> To: logistics at lists.fedoraproject.org; simon at zikula.org
>> Subject: Re: Blockers
>>
>> Simon -
>>
>> Where is the patch you created?
>>
>> -Robyn
>>
>> On Wed, Apr 21, 2010 at 8:26 AM, Hiemanshu Sharma<hiemanshu at gmail.com>
>> wrote:
>>      
>>> On 04/21/2010 08:41 PM, Paul W. Frields wrote:
>>>        
>>>> On Wed, Apr 21, 2010 at 11:26:24AM +0530, Hiemanshu Sharma wrote:
>>>>
>>>>          
>>>>> On 04/21/2010 05:01 AM, Robyn Bergeron wrote:
>>>>>
>>>>>            
>>>>>> I just filed this:
>>>>>>
>>>>>> #2111: Logging into Fedora Insight when not in cmsadmin group
>>>>>>              
>> gives 403 error
>>      
>>>>>> ----------------------+-------------------------------------------
>>>>>>              
>> ----------
>>      
>>>>>>     Reporter:  rbergero  |       Owner:  itbegins
>>>>>>        Type:  bug       |      Status:  new
>>>>>>     Priority:  blocker   |   Milestone:
>>>>>> Component:  General   |     Version:  Test
>>>>>>     Severity:  High      |    Keywords:  insight zikula
>>>>>> ----------------------+-------------------------------------------
>>>>>>              
>> ----------
>>      
>>>>>>     = phenomenon =
>>>>>>     When logging into zikula / FI on pt6 as new users "robynztest",
>>>>>>              
>> get the
>>      
>>>>>>     following error:
>>>>>>     ----
>>>>>>     Access denied (error 403)
>>>>>>     Sorry! You don't have authorisation for the page you wanted.
>>>>>>     Additional information
>>>>>>     Sorry! You have not been granted access to this page.
>>>>>>
>>>>>>     ----
>>>>>>     Simon posted an email to the logistics list this afternoon
>>>>>>              
>> saying there is
>>      
>>>>>>     a patch -
>>>>>>     http://lists.fedoraproject.org/pipermail/logistics/2010-
>>>>>>              
>> April/000500.html
>>      
>>>>>>     That email posted approximately the same time that
>>>>>>              
>> Hiemanshu/Gwerra posted
>>      
>>>>>>     that he thought he had fixed the problem.  I'm not sure if
>>>>>>              
>> Hiemanshu
>>      
>>>>>>     applied the patch, or something else.
>>>>>>
>>>>>>     At this point: users not in cmsadmin no longer have
>>>>>>              
>> administrative access
>>      
>>>>>>     when they log in - but they are not able to log in past a 403
>>>>>>              
>> error.   The
>>      
>>>>>>     new users do get added to the "users" group - and not the
>>>>>>              
>> "administrators"
>>      
>>>>>>     group - so we need to figure out why they're getting a 403.
>>>>>>
>>>>>>              
>>>>> Well I did this patch last night while I was working on it, I made
>>>>>            
>> the
>>      
>>>>> patches on my own (from what ever was on pt6.) I am still working
>>>>>            
>> on it,
>>      
>>>>> hopefully I should have it figured out soon.
>>>>>
>>>>>            
>>>> So I'm not sure -- does this mean, Hiemanshu, that you wrote your
>>>> patches on top of Simon's code, or backed his up, or... ?
>>>>
>>>> Can we find the patch Simon did on a list or a ticket, so that we
>>>>          
>> can
>>      
>>>> reapply it as needed?
>>>>
>>>>
>>>>          
>>> Whatever changes I made were made over simons code on pt6. I am not
>>>        
>> sure
>>      
>>> whether it was the old code or the patched code.
>>>
>>> - Hiemanshu
>>> _______________________________________________
>>> logistics mailing list
>>> logistics at lists.fedoraproject.org
>>> https://admin.fedoraproject.org/mailman/listinfo/logistics
>>>
>>>        
>> No virus found in this incoming message.
>> Checked by AVG - www.avg.com
>> Version: 9.0.801 / Virus Database: 271.1.1/2823 - Release Date:
>> 04/21/10 12:09:00
>>      
> _______________________________________________
> logistics mailing list
> logistics at lists.fedoraproject.org
> https://admin.fedoraproject.org/mailman/listinfo/logistics
>