[Owncloud] Bookmarks app in 4.0.2

Arthur Schiwon blizzz at owncloud.com
Wed Jun 20 17:54:46 UTC 2012


Hey,

On 06/20/2012 05:55 PM, Victor Dubiniuk wrote:
> Hi,
>
> this one seems to be a victim of CSRF protection. As a temporary fix
>
> apps/bookmarks/addBm.php, line #31
>
> addBookmark($_POST['url'], '', 'Read-Later');
>
>
> should be changed to
>
> addBookmark($_GET['url'], '', 'Read-Later');

i'm looking into it now, thanks for the hint Victor.

The bug report in question is 
http://bugs.owncloud.org/thebuggenie/owncloud/issues/oc-1009 btw.

Cheers
Arthur

>
>
>
> P.S. Please do not forget to submit it to the bug tracker.
>
> ---
> Victor Dubiniuk
>
> On Wed, Jun 20, 2012 at 6:23 PM, Christian Reiner<
> foss at christian-reiner.info>  wrote:
>
>> On Wednesday 20 June 2012 15:38:56 Christian Homrighausen wrote:
>>> Hi,
>>>
>>> Whenever I use the "read later" bookmarklet to create a bookmark, there
>>> is only an "unnamed" entry with no URL.
>>
>> I confirm this bug, using OC-4.0.2 under opensuse x86, apache 2.2, php 5.3.
>> Firefox 13 under opensuse x86-64 on the client side.
>>
>> In addition I observe this trange behaviour:
>> instead of calling the bookmark app in a separate window / tab when
>> clicking
>> the bookmark a mini window is opened and I get the raw looking text
>> "Saved!".
>> Huge black letters on apure white background. And the text is too big for
>> that
>> window. Looks like something is really broken.
>>
>> Since the Shortlet from the Shorty app works as expected, and since that
>> uses
>> the same strategy, we can say:
>> it is the bookmark app broken under OC-4.0.2, it is not an issue with the
>> browser or anything.
>>
>> Christian Reiner
>> _______________________________________________
>> Owncloud mailing list
>> Owncloud at kde.org
>> https://mail.kde.org/mailman/listinfo/owncloud
>>
>
>
>
> _______________________________________________
> Owncloud mailing list
> Owncloud at kde.org
> https://mail.kde.org/mailman/listinfo/owncloud



More information about the Owncloud mailing list