question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Pinning integration: descope stats from Settings screen

See original GitHub issue

Initial Settings mockups in WebUI include columns with basic stats: utilized storage as “Files” and “Bandwidth Used”:

files-storage-

Problem

Pinning Service API spec does not provide endpoint for reading this information.

Additionally, after sync with @jacobheun @aschmahmann and @alanshaw we decided to make API secrets write-only in go/js-ipfs config for security reasons. This means WebUI won’t be able to read API secret and make remote calls on its own.

Solutions

We have two ways of tackling this:

  • weeks: add /stats endpoint to Pinning Service API and then support for it needs to land in go/js-ipfs and webui

or

  • hours: descope this from MVP and remove “Files” and “Bandwidth Used” columns from Settings screen. User will still be able to check stats via three dot menu and a link to pinning service website.

Suggested solution

I don’t believe vaue added by those columns is worth additional complexity and additional work on Pinning Service and Core Impl. WG ends – are we ok with descoping them from MVP?

cc @jacobheun @jessicaschilling @rafaelramalho19 for quick temperature check, before I escalate this upstream

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Comments:8 (7 by maintainers)

github_iconTop GitHub Comments

1reaction
jessicaschillingcommented, Jul 16, 2020

+1 for “pin count” being a good usage signal, at least for this version of the API. I suspect some others will emerge in time as more pinning services come on as adopters.

1reaction
lidelcommented, Jul 15, 2020

Context: we discussed this briefly during yesterday’s review call. My takeaway was we are ok with removing those specific storage/bandwidth stats, as long we replace them with something that provides “usage signal” to the user, like @momack2 said.

Potential candidate is “pin count” – number coud be read from count field returned by GET /pins?status=pinned in Pinning Service API (but we need to figure out how to expose this value over go-ipfs’ HTTP API).

Separately, we could discuss adding PinStatus.size field to indicate the size pinned DAG uses at the pinning service, but it brings similar question on how to expose it to WebUI from go-ipfs.

@jacobheun where should we track mapping from Pinning Service API to go-ipfs commands and HTTP API? Even a hackpad would help with resolving gaps like ones described above.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Pinning integration: descope stats from Settings screen #1544
Initial Settings mockups in WebUI include columns with basic stats: utilized storage as "Files" and "Bandwidth Used": Problem Pinning ...
Read more >
Certificate and Public Key Pinning | OWASP Foundation
Certificate and Public Key Pinning is a technical guide to implementing certificate and public key pinning as discussed at the Virginia chapter's ...
Read more >
Qualtrics XM App
The Qualtrics XM App lets you and your colleagues view your dashboards on the go. With this app, you can view widgets, filter...
Read more >
SSL Pinning Settings - VMware Docs
With the SSL Pinning settings page, you can: Activate or deactivate the SSL Pinning for the organization group. Upload the Device Services ...
Read more >
HP Application Lifecycle Management (ALM) User Guide
use the ALM open test architecture to integrate your own configuration management, defect tracking, and home-grown testing tools with an ALM project.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found