status-im / status-desktop

Status Desktop client made in Nim & QML
https://status.app
Mozilla Public License 2.0
287 stars 78 forks source link

Pass `status-go` Version to `status-go` in Desktop App #16138

Open Samyoul opened 4 weeks ago

Samyoul commented 4 weeks ago

Description:

It appears that the Desktop app currently has a concept of the status-go version, but this information may not be passed to status-go itself. In the mobile app, we utilize a status-go-version.json file to communicate the version to status-go. However, it seems that the Desktop app does not have a similar mechanism in place.

image

Current Behavior:

image

Expected Behavior:

Steps to Reproduce:

  1. Inspect the current version handling in the Desktop app.
  2. Note that the status-go version is not communicated to status-go.

You can also inspect the proxy server access logs and see that the status-go version is missing from the RPC client User Agent version: image

Additional Context:

Relevant Links:

Proposal:

Implement a mechanism in the Desktop app to pass the status-go version to status-go, ensuring consistency across platforms. This could involve creating a similar JSON file or integrating the version information in another suitable way.

Samyoul commented 3 weeks ago

I've just noticed this:

image

A version of the Desktop app is declaring status-go version 0.179.11, that is the version value of the VERSION file in the status-go repo . See here https://github.com/status-im/status-go/blob/develop/VERSION

caybro commented 3 weeks ago

I've just noticed this:

image

A version of the Desktop app is declaring status-go version 0.179.11, that is the version value of the VERSION file in the status-go repo . See here https://github.com/status-im/status-go/blob/develop/VERSION

That looks like either the release branch, or an older master branch. It's no longer the case, we use this to get the status-go version number (from Makefile):

STATUSGO_VERSION=`(cd vendor/status-go; git describe --tags --abbrev=0)`

which we then display in the Settings/About page. It's not used for any other purpose afaik