Shields.io no puede recuperar la API de estadísticas de Discourse

haikalpribadi · 14 Septiembre, 2019 13:25

Hola a todos,

Hemos tenido un escudo de Discourse en nuestro repositorio durante un tiempo, y recientemente dejó de funcionar.

Si vas a 404: badge not found y seleccionas cualquier escudo de Discourse, puedes ingresar la dirección de tu dominio de Discourse y te mostrará el escudo con las estadísticas correctas. Puedes probar esto con meta.discourse.org.

Sin embargo, cuando ingresamos la dirección de nuestro host de Discourse (https://discuss.grakn.ai), para cualquier estadística y tanto para http como https, siempre devuelve “invalid”.

Cuando un host no se encuentra, Shields.io devolvería “inaccessible”. Por lo tanto, asumimos que “invalid” significa que es accesible pero hay problemas de derechos de acceso o respuestas inválidas.

¿Es posible que una actualización reciente haya roto algo en la API de estadísticas de Discourse que utiliza Shields.io?

¡Muchas gracias!

marianord · 15 Septiembre, 2019 03:00

En mi sitio funciona correctamente. Quizás no estás configurando el protocolo correctamente, o la instalación de Grakn en Discourse tiene alguna modificación que rompe ese endpoint.

gerhard · 15 Septiembre, 2019 10:35

Quizás quieras consultar el problema con Shields.io. Funciona con todos los demás sitios que probé, así que este no es un error nuestro.

haikalpribadi · 15 Septiembre, 2019 19:20

@marianord esa es exactamente mi pregunta: ¿dónde están los “protocolos” que mencionas? ¿Cómo se pueden configurar? No he cambiado ninguna configuración.

@gerhard dado que shields.io funciona para otros sitios de Discourse, no parece probable que sea un problema de su parte. A menos que no estén leyendo correctamente la salida de nuestras estadísticas del sitio, pero ¿cómo podemos averiguar esto? ¿Cuál es el punto de conexión de Discourse que se usa para consultar las estadísticas? ¿Quizás deberíamos empezar por ahí?

marianord · 15 Septiembre, 2019 19:27

Estoy mencionando http frente a https.

max_grakn · 19 Septiembre, 2019 11:05

Esto ocurrió porque nuestra instalación de Discourse bloqueó el agente de usuario de Shields.io (Shields.io). Esta configuración se llama agentes de usuario de rastreadores en la lista blanca y se puede editar en
<discourse_server>/admin/site_settings/category/all_results?filter=crawler

haikalpribadi · 19 Septiembre, 2019 11:18

¡Interesante! Gracias @max_grakn. Agregamos recientemente a Googlebot a la lista blanca; creo que eso podría ser la causa.

@codinghorror, ¿se supone que debemos usar la lista negra y la lista blanca al mismo tiempo? Es decir, si agregas cosas a la lista blanca, ¿significa que todo lo demás está en la lista negra (lo que haría redundante la lista negra)?

codinghorror · 19 Septiembre, 2019 12:41

No, la lista blanca de rastreadores es muy peligrosa y solo debe usarse con precaución, según el texto de ayuda.

Agentes de usuario de los rastreadores web que deben tener permitido acceder al sitio. ¡ADVERTENCIA! AL CONFIGURAR ESTO, SE DENEGARÁ EL ACCESO A TODOS LOS RASTREADORES QUE NO ESTÉN INCLUIDOS EN ESTA LISTA!

barto_95 · 13 Noviembre, 2020 10:16

Hola, tengo el mismo problema cuando recibo datos inválidos en una prueba… ¿tienes alguna idea?

Ahora está bien, es necesario activar las estadísticas anónimas en:

Admin → configuración → otro → compartir estadísticas anonimizadas = Habilitado

y ahora funciona correctamente para shields.io

spdegabrielle · 15 Noviembre, 2023 18:22

Tengo el mismo problema, pero las estadísticas anónimas ya están habilitadas

[![Racket Discourse](upload://6fa5jbSn04vRLXdubAYmFJt5emf.svg)]

[]

Arkshine · 15 Noviembre, 2023 19:41

Relacionado con:

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

Es porque algunos campos han sido renombrados en su forma plural.

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

Alguien necesitará modificar el código aquí para que recurra a la forma plural:

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 · 20 Noviembre, 2023 15:52

Esto sigue roto. ¿Funciona para alguien más?

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

BryanV · 6 Diciembre, 2023 17:35

Se abrió una PR hace unos días:

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.

Pero, naturalmente, este cambio supone una carga para shields.io para admitir ambas versiones (a perpetuidad, supongo) ya que no hay garantía de que cualquier instancia de Discourse se haya actualizado o no, por lo que no es tan simple como cambiar a plurales.

Tema		Respuestas	Vistas
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	13 Septiembre 2019
Discourse CDNs are blocked by privacy badger Site feedback	47	12925	15 Agosto 2017
Can I use the Discourse API on an officially-hosted instance? Support	1	356	9 Julio 2021
Access reports JSON data via Pipedream Development rest-api	2	853	28 Julio 2021
Interact with discourse from Python? Development	32	5294	20 Abril 2026

Shields.io no puede recuperar la API de estadísticas de Discourse

Temas relacionados