Shields.io incapable de récupérer l'API de statistiques Discourse

haikalpribadi · Septembre 14, 2019, 1:25

Bonjour à tous,

Nous avons un badge Discourse sur notre dépôt depuis un moment, mais il a récemment cessé de fonctionner.

Si vous allez sur 404: badge not found et sélectionnez n’importe quel badge Discourse, vous pouvez entrer l’adresse de votre domaine Discourse et le badge s’affichera avec les statistiques correctes. Vous pouvez essayer cela avec meta.discourse.org.

Cependant, lorsque nous saisissons l’adresse de notre hôte Discourse (https://discuss.grakn.ai), pour toutes les statistiques et pour le http/https, il renvoie toujours « invalid ».

Lorsqu’un hôte n’est pas trouvé, Shields.io renverrait « inaccessible ». Nous supposons donc que « invalid » signifie qu’il est accessible mais qu’il y a des problèmes de droits d’accès ou des réponses invalides.

Est-il possible qu’une mise à jour récente ait cassé quelque chose dans l’API de statistiques de Discourse que Shields.io utilise ?

Merci beaucoup !

marianord · Septembre 15, 2019, 3:00

Cela fonctionne de mon côté sur mon site. Peut-être que vous n’avez pas correctement défini le protocole ? Ou alors Grakn Discourse comporte une modification qui rompt ce point de terminaison.

gerhard · Septembre 15, 2019, 10:35

Vous devriez poser cette question à Shields.io. Cela fonctionne avec tous les autres sites que j’ai testés, donc ce n’est pas un bug de notre côté.

haikalpribadi · Septembre 15, 2019, 7:20

@marianord c’est exactement ma question : où se trouvent les « protocoles » dont vous parlez ? Comment peuvent-ils être configurés ? Je n’ai modifié aucun paramètre.

@gerhard étant donné que shields.io fonctionne pour d’autres sites Discourse, il est peu probable qu’il s’agisse d’un problème de leur côté. À moins qu’ils ne lisent pas correctement la sortie de nos statistiques de site — mais comment pouvons-nous le vérifier ? Quel est le point de terminaison de Discourse utilisé pour interroger les statistiques ? Peut-être devrions-nous commencer par là ?

marianord · Septembre 15, 2019, 7:27

Je fais référence à http par rapport à https.

max_grakn · Septembre 19, 2019, 11:05

Cela est dû au fait que notre installation de Discourse a bloqué l’agent utilisateur de Shields.io (Shields.io). Ce paramètre s’appelle agents utilisateur de robots d’indexation autorisés et peut être modifié à l’adresse suivante :
<discourse_server>/admin/site_settings/category/all_results?filter=crawler

haikalpribadi · Septembre 19, 2019, 11:18

Intéressant ! Merci @max_grakn ! Nous avons récemment ajouté Googlebot à la liste blanche, je pense que cela pourrait en être la cause.

@codinghorror, sommes-nous censés utiliser la liste noire et la liste blanche en même temps ? Autrement dit, si vous ajoutez des éléments à la liste blanche, cela signifie-t-il que tout le reste est automatiquement placé sur la liste noire (rendant ainsi la liste noire redondante) ?

codinghorror · Septembre 19, 2019, 12:41

Non, la liste blanche des robots d’indexation est très dangereuse et ne doit être utilisée qu’avec précaution, comme l’indique le texte d’aide.

Agents utilisateurs des robots d’indexation autorisés à accéder au site. ATTENTION ! LA CONFIGURATION DE CETTE OPTION INTERDIT TOUS LES ROBOTS D’INDEXATION NON INSCRITS ICI !

barto_95 · Novembre 13, 2020, 10:16

Bonjour, je rencontre le même problème lorsqu’un test reçoit des données invalides… avez-vous une idée ?

C’est résolu maintenant, il est nécessaire d’activer les statistiques anonymes dans :

Admin → paramètres → autres → partager des statistiques anonymisées = Activé

et maintenant, c’est bon pour shields.io

spdegabrielle · Novembre 15, 2023, 6:22

J’ai le même problème, mais le partage de statistiques anonymes est déjà activé

[]

Arkshine · Novembre 15, 2023, 7:41

Lié à :

github.com/discourse/discourse

DEV: Ability to collect stats without exposing them via API

main ← dev/ability-to-collect-stats-without-public-exposing

opened 06:37PM - 13 Oct 23 UTC

AndrewPrigorshnev

+228 -119

This adds the ability to collect stats without exposing them among other stats v…ia API. The most important thing I wanted to achieve is to provide an API where stats are not exposed by default, and a developer has to explicitly specify that they should be exposed (`expose_via_api: true`). Implementing an opposite solution would be simpler, but that's less safe in terms of potential security issues. When working on this, I had to refactor the current solution. I would go even further with the refactoring, but the next steps seem to be going too far in changing the solution we have, and that would also take more time. Two things that can be improved in the future: 1. Data structures for holding stats can be further improved 2. Core stats are hard-coded in the About template (it's hard to fix it without correcting data structures first, see point 1): https://github.com/discourse/discourse/blob/63a0700d45f755d0f432a9075ae7afbed9cd6ab0/app/views/about/index.html.erb#L61-L101 The most significant refactorings are: 1. Introducing the `Stat` model 2. Aligning the way the core and the plugin stats' are registered

C’est parce que quelques champs ont été renommés sous leur forme plurielle.

topic_count → topics_count
post_count → posts_count
user_count → users_count
like_count → likes_count

Quelqu’un devra modifier le code ici pour revenir à la forme plurielle :

github.com/badges/shields

services/discourse/discourse.service.js

master

import Joi from 'joi'
import { metric } from '../text-formatters.js'
import { nonNegativeInteger, url } from '../validators.js'
import { BaseJsonService, queryParams } from '../index.js'

const schemaSingular = Joi.object({
  topic_count: nonNegativeInteger,
  user_count: nonNegativeInteger,
  post_count: nonNegativeInteger,
  like_count: nonNegativeInteger,
}).required()

const schemaPlural = Joi.object({
  topics_count: nonNegativeInteger,
  users_count: nonNegativeInteger,
  posts_count: nonNegativeInteger,
  likes_count: nonNegativeInteger,
})

const schema = Joi.alternatives(schemaSingular, schemaPlural)

This file has been truncated. show original

spdegabrielle · Novembre 20, 2023, 3:52

Ceci est toujours cassé - est-ce que ça marche pour quelqu’un d’autre ?

![Racket Discourse](upload://7asTK98zfWLXRebm11uEN3KPM5N.svg)

BryanV · Décembre 6, 2023, 5:35

Une PR a été ouverte il y a quelques jours :

github.com/badges/shields

[Discourse] Update schema keys to use plural form (`topic_count` -> `topics_count`)

master ← joshuacwnewton:jn/9776-fix-discourse-js-schema

opened 08:16PM - 03 Dec 23 UTC

joshuacwnewton

+110 -71

This PR does a simple find and replace to update the Discourse schema to use the… current keys found in https://meta.discourse.org/site/statistics.json, as the older keys now return "Invalid response". > [!NOTE] > I was wondering, though, if the old keys should still be preserved, as to not break backwards compatibility with servers using the old names. > > (I tried finding the source of the change in Discourse's release notes/changelog, but there's surprisingly little information when querying e.g. "statistics.json" or "topic_count"/"topics_count". So, I'm not sure when this change actually took place, and thus it's hard to estimate the likelihood of there being servers out there using the old keys.) > > If we do want to support both sets of keys, then I'm not 100% sure what to do, as I've never written JavaScript before. (I tried reading the Joi docs, and got as far as using `.or()` in the schema definition for each pair of keys + using a `try`/`catch` when creating the `metricIntegrations` object, but I wasn't confident enough to commit that solution before discussing it first.) Fixes #9776.

Mais naturellement, ce changement impose à shields.io de supporter les deux versions (à perpétuité, je suppose) car il n’y a aucune garantie qu’une instance Discourse donnée ait été mise à jour ou non, donc ce n’est pas aussi simple que de passer au pluriel.

Sujet		Réponses	Vues
Is there a nice GH badges (ala shields.io images) design to put in README to link from GitHub to our hosted discourse? Feature	17	5149	Septembre 13, 2019
Discourse CDNs are blocked by privacy badger Site feedback	47	12925	Août 15, 2017
Can I use the Discourse API on an officially-hosted instance? Support	1	356	Juillet 9, 2021
Access reports JSON data via Pipedream Development rest-api	2	853	Juillet 28, 2021
Interact with discourse from Python? Development	32	5295	Avril 20, 2026

Shields.io incapable de récupérer l'API de statistiques Discourse

Sujets connexes