Shields.io не может получить статистику API Discourse

haikalpribadi · 14.Сентябрь.2019 13:25:54

Всем привет,

У нас уже какое-то время был бейдж Discourse в нашем репозитории, но недавно он перестал работать.

Если перейти на 404: badge not found и выбрать любой бейдж Discourse, можно ввести адрес домена вашего форума, и он отобразит бейдж с правильными статистическими данными. Вы можете попробовать это с meta.discourse.org.

Однако, когда мы вводим адрес нашего хоста (https://discuss.grakn.ai), для любых статистических данных и для обоих протоколов http/https всегда возвращается ошибка «invalid».

Если хост не найден, Shields.io возвращает «inaccessible». Таким образом, мы предполагаем, что «invalid» означает, что хост доступен, но есть проблемы с правами доступа или некорректные ответы.

Возможно, недавнее обновление или обновление сломало что-то в API статистики Discourse, которое использует Shields.io?

Спасибо большое!

marianord · 15.Сентябрь.2019 03:00:43

У меня на сайте всё работает. Возможно, вы неправильно указываете протокол? Или же в Grakn Discourse есть какая-то модификация, нарушающая работу этого конечного узла.

gerhard · 15.Сентябрь.2019 10:35:22

Возможно, стоит обратиться к Shields.io по поводу этой проблемы. На всех остальных сайтах, которые я тестировал, всё работает, так что это не наша ошибка.

haikalpribadi · 15.Сентябрь.2019 19:20:54

@marianord, это именно мой вопрос: где находятся «протоколы», о которых вы говорите? Как их настроить? Я не менял никаких настроек.

@gerhard, учитывая, что shields.io работает для других сайтов на Discourse, маловероятно, что проблема на их стороне. Если только они не правильно не обрабатывают вывод статистики нашего сайта — но как нам это выяснить? Какой конечный пункт Discourse используется для запроса статистики? Возможно, стоит начать именно с этого?

marianord · 15.Сентябрь.2019 19:27:23

Я упоминаю http против https.

max_grakn · 19.Сентябрь.2019 11:05:56

Это произошло из-за того, что наша установка Discourse заблокировала пользовательский агент Shields.io (Shields.io). Этот параметр называется Разрешённые пользовательские агенты поисковых роботов и может быть изменён по адресу
<discourse_server>/admin/site_settings/category/all_results?filter=crawler

haikalpribadi · 19.Сентябрь.2019 11:18:05

Интересно! Спасибо, @max_grakn! Мы недавно добавили Googlebot в белый список, и, думаю, это может быть причиной.

@codinghorror, должны ли мы использовать чёрный и белый списки одновременно? То есть, если вы добавляете элементы в белый список, означает ли это, что всё остальное автоматически попадает в чёрный список (и тем самым делает чёрный список избыточным)?

codinghorror · 19.Сентябрь.2019 12:41:39

Нет, белый список для краулеров очень опасен, и его следует использовать с осторожностью в соответствии с текстом справки.

User agents веб-краулеров, которым следует разрешить доступ к сайту. ВНИМАНИЕ! УКАЗАНИЕ ЭТОГО СПИСКА ЗАПРЕТИТ ДОСТУП ВСЕХ КРАУЛЕРОВ, НЕ ВКЛЮЧЁННЫХ В НЕГО!

barto_95 · 13.Ноябрь.2020 10:16:19

Привет, у меня возникла та же проблема: я получил некорректные данные… Есть какие-то идеи?

Теперь всё в порядке. Необходимо включить анонимную статистику в:

Админ → Настройки → Другое → Обмен анонимизированной статистикой = Включено

Теперь с shields.io всё работает.

spdegabrielle · 15.Ноябрь.2023 18:22:58

У меня та же проблема, но «Отправлять анонимную статистику» уже включено

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

[]

Arkshine · 15.Ноябрь.2023 19:41:54

Связано с:

github.com/discourse/discourse

DEV: Ability to collect stats without exposing them via API (#23933)

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

merged 08:44PM - 09 Nov 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

Это произошло потому, что несколько полей были переименованы во множественном числе.

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

Кто-то должен будет изменить код здесь, чтобы предусмотреть возврат к форме множественного числа:

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.Ноябрь.2023 15:52:44

Это всё ещё не работает — у кого-то ещё это функционирует?

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

BryanV · 06.Декабрь.2023 17:35:34

Несколько дней назад был открыт PR:

github.com/badges/shields

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

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

merged 08:00PM - 06 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.

Но, разумеется, это изменение возлагает на shields.io бремя поддержки обеих версий (навсегда, полагаю), так как нет гарантии, что какой-либо конкретный экземпляр Discourse был обновлён или нет, поэтому всё не так просто, как переход к множественному числу.

Тема		Ответов	Просм.
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	5117	13.09.2019
Discourse CDNs are blocked by privacy badger Site feedback	47	12907	15.08.2017
Can I use the Discourse API on an officially-hosted instance? Support	1	350	09.07.2021
Access reports JSON data via Pipedream Development rest-api	2	845	28.07.2021
Interact with discourse from Python? Development	32	5218	20.04.2026

Shields.io не может получить статистику API Discourse

Связанные темы