Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id DBB80200C40 for ; Thu, 23 Mar 2017 22:05:17 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id DA325160B83; Thu, 23 Mar 2017 21:05:17 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 2B82C160B68 for ; Thu, 23 Mar 2017 22:05:17 +0100 (CET) Received: (qmail 99307 invoked by uid 500); 23 Mar 2017 21:05:16 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 99274 invoked by uid 99); 23 Mar 2017 21:05:14 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 23 Mar 2017 21:05:14 +0000 Received: from mail-it0-f45.google.com (mail-it0-f45.google.com [209.85.214.45]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id 8E0AD1A0701 for ; Thu, 23 Mar 2017 21:05:14 +0000 (UTC) Received: by mail-it0-f45.google.com with SMTP id w124so47779548itb.1 for ; Thu, 23 Mar 2017 14:05:14 -0700 (PDT) X-Gm-Message-State: AFeK/H2PwzBiSAMnGGfB8icgk6AJDjnA4O7rrPt7BiHPo2i+kg+Sv4xKzUbxe91Yml30H9bI1mzkmrmgjm7Oeg== X-Received: by 10.36.33.73 with SMTP id e70mr4718673ita.9.1490303113736; Thu, 23 Mar 2017 14:05:13 -0700 (PDT) MIME-Version: 1.0 Received: by 10.36.196.133 with HTTP; Thu, 23 Mar 2017 14:04:33 -0700 (PDT) In-Reply-To: References: From: Shazron Date: Thu, 23 Mar 2017 14:04:33 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [DISCUSS] Plugin search is broken To: Shazron Cc: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=001a1147c5beb3be9c054b6c3b6a archived-at: Thu, 23 Mar 2017 21:05:18 -0000 --001a1147c5beb3be9c054b6c3b6a Content-Type: text/plain; charset=UTF-8 I'm going to assume by lazy consensus that moving off npmsearch is a 'go'. Filed: https://issues.apache.org/jira/browse/CB-12602 Currently our plugin search works after I filed those two issues in their repo and they were was fixed, thus fixing our search. On Sun, Mar 19, 2017 at 4:36 PM, Shazron wrote: > Consequently, found another bug and filed: https://issues.apache. > org/jira/browse/CB-12585 > > On Sun, Mar 19, 2017 at 4:04 PM, Shazron wrote: > >> I filed https://github.com/nodesource/npmsearch/issues/54 earlier and >> the dev fixed it right away, yay! So currently our search is working again, >> but the original proposal stands regarding migration. >> >> On Sun, Mar 19, 2017 at 1:50 PM, Shazron wrote: >> >>> Ignore the last message ;) >>> >>> This results in zero results. >>> http://npmsearch.com/query/?fields=name,keywords,license,des >>> cription,author,modified,homepage,version,rating&q=keywords: >>> %22ecosystem:cordova%22&sort=rating:desc >>> >>> The same URL without the sort query param has results: >>> >>> http://npmsearch.com/query/?fields=name,keywords,license,des >>> cription,author,modified,homepage,version,rating&q=keywords: >>> %22ecosystem:cordova%22 >>> >>> >>> On Sun, Mar 19, 2017 at 1:44 PM, Shazron wrote: >>> >>>> I believe the original problem lies in npmsearch.com erroring on a >>>> query param "sort": >>>> http://npmsearch.com/query/?sort=rating:desc&fields=name,key >>>> words,license,description,author,modified,homepage,version,r >>>> ating&sort=&q=keywords:%22ecosystem:cordova%22 >>>> >>>> The URL above is a query that we send to it. (a simple "sort=" results >>>> in the same error) >>>> >>>> If "sort" is removed, it is fine: >>>> http://npmsearch.com/query/?fields=name,keywords,license,des >>>> cription,author,modified,homepage,version,rating&sort=&q=key >>>> words:%22ecosystem:cordova%22 >>>> >>>> >>>> >>>> >>>> >>>> >>>> On Sun, Mar 19, 2017 at 1:25 PM, Shazron wrote: >>>> >>>>> The impetus for this is, users are reporting [1][2][3][4] that they >>>>> can't find plugins, and I don't think they know of a search alternative. >>>>> >>>>> In the near term, I think we should tweet how to search for plugins >>>>> using npmjs.com >>>>> >>>>> [1] https://twitter.com/edeortuzar/status/843310576101146625 >>>>> [2] https://twitter.com/kumakumadev/status/843149557127155712 >>>>> [3] https://twitter.com/quangv/status/842925319254360064 >>>>> [4] https://twitter.com/skumarSDsoft/status/842677001433497600 >>>>> >>>>> >>>>> >>>>> >>>>> On Sun, Mar 19, 2017 at 1:19 PM, Shazron wrote: >>>>> >>>>>> ### PROBLEM >>>>>> >>>>>> Plugin search is broken. >>>>>> >>>>>> ### DESCRIPTION >>>>>> >>>>>> Page: >>>>>> http://cordova.apache.org/plugins/ >>>>>> >>>>>> JIRA: >>>>>> https://issues.apache.org/jira/browse/CB-12579 >>>>>> >>>>>> >>>>>> Q: How does the plugin search work? >>>>>> >>>>>> A: >>>>>> Page Source: http://cordova.apache.org/plugins >>>>>> and the source (this specific line): http://svn.apache.org/v >>>>>> iewvc/cordova/site/public/plugins/index.html?view=markup#l117 >>>>>> >>>>>> which is added from here: >>>>>> https://github.com/apache/cordova-docs/blob/master/www/plugi >>>>>> ns/index.html >>>>>> >>>>>> which is generated from this React code: >>>>>> https://github.com/apache/cordova-docs/blob/a3589ce66336821f >>>>>> 2fd714a1115125cfcfa9fd1b/www/static/plugins/app.js >>>>>> >>>>>> Nothing shows up as errors in the Web Inspector, so it is hard to >>>>>> track down. >>>>>> >>>>>> ### MY PROPOSAL >>>>>> >>>>>> The era of needing npmsearch.com to populate this page is over. We >>>>>> could just redirect users to npmjs.com and add the >>>>>> "ecosystem:cordova" token (i.e https://www.npmjs.com/sea >>>>>> rch?q=ecosystem%3Acordova) >>>>>> >>>>>> The alternative is to use npm's Registry API: >>>>>> https://github.com/npm/registry/commits/master/docs/REGISTRY-API.md >>>>>> >>>>>> The API was only documented earlier this year, if not we would have >>>>>> used it. >>>>>> >>>>>> >>>>> >>>> >>> >> > --001a1147c5beb3be9c054b6c3b6a--